Stream: Covid-19 Response
Topic: How many resources at once
Michael Donnelly (May 15 2020 at 01:12):
I've got searches working for multiple locations (including leaving location out and getting all locations). If you do a search for the last two months for the CSC Measure for all locations, it has a lot of results. The first thousand take more than 30 seconds and weigh in at more than 5 MB.
Michael Donnelly (May 15 2020 at 01:13):
Is 30 seconds too long? (Indicate with thumbs up / thumbs down)
Michael Donnelly (May 15 2020 at 01:13):
Is 5 MB too big? (same)
Michael Donnelly (May 15 2020 at 01:14):
If need be, I can set a smaller limit than the base limit of 1,000.
Abbie Watson (May 15 2020 at 01:52):
The 5MB size isn't a problem, it's the number of records and the memory references to them (and subsequent garbage collection and memory leaks). I vote for 200 or 500 records.
Also, does your server support count
? There's a URL parameter for specifying the number of records to be returned. Worth encouraging in the IG.
Michael Donnelly (May 15 2020 at 01:53):
Sure does.
Michael Donnelly (May 15 2020 at 01:53):
If you feel like it, feel free to hit the server and see how memory is for different numbers.
Abbie Watson (May 15 2020 at 01:54):
I'd be inclined to provide that as a UI option, and let the user decide how many records they want to display. And pass that preference along in the count
parameter.
Michael Donnelly (May 15 2020 at 01:54):
Michael Donnelly (May 15 2020 at 01:54):
Makes sense.
Abbie Watson (May 15 2020 at 01:54):
Perfect. Include in the report-out and IG updates?
Michael Donnelly (May 15 2020 at 01:55):
Wilco.
Michael Donnelly (May 15 2020 at 01:58):
(We do appear to have an escaping bug in the next link on the second page for the third page. Looking into that now....)
Abbie Watson (May 15 2020 at 02:02):
Heh. Computers.
FYI... I'm wrapping up the Gravity track. Apparently nobody else showed up with a PatientApp or ScreeningApp, so I was the only person the PMEHR folks had to connectathon with. Wrapping that up, and then deploying a patch to support the Push usecase on Inferno, and should have the Intermediary/Hub functionality in place.
Michael Donnelly (May 15 2020 at 02:02):
Right on.
Michael Donnelly (May 15 2020 at 02:02):
They're lucky you showed up, then.
Abbie Watson (May 15 2020 at 02:06):
Thx. I appreciate that.
Michael Donnelly (May 18 2020 at 16:42):
Abigail Watson said:
Perfect. Include in the report-out and IG updates?
https://github.com/AudaciousInquiry/fhir-saner/issues/84
Last updated: Apr 12 2022 at 19:14 UTC