DataBiosphere / data-browser

Apache License 2.0
11 stars 4 forks source link

Data Browser causes Azul to make redundant TDR requests #3142

Open hannes-ucsc opened 2 years ago

hannes-ucsc commented 2 years ago

Data Browser makes concurrent requests against /index and /summary endpoints and that kicks off two simultaneous requests against TDRs enumerateSnapshots endpoint. By staggering the /summary request to occur after the /index request returns, we can eliminate that request and halve the load on TDR. It would also speed up the /index response (since it won't be competing with the concurrent /summary request) and cause a better, more responsive UX. Sometimes serializing things actually makes reduces the overall time needed. This is one of those cases.

For background see here:

https://ucsc-gi.slack.com/archives/C705Y6G9Z/p1666748331379769?thread_ts=1666724068.705709&cid=C705Y6G9Z

hannes-ucsc commented 1 year ago

This has been in the sprint column for ages. Is this actually being worked on? And why is the estimate 0?

theathorn commented 1 year ago

There is nothing incorrect about the current Data Browser behavior, but the assumption is that a simple change "staggering the /summary request after the /index request (on a per user basis), and rendering the table when the /index request returns should improve the UX on initial visits". Adding request synchronization within the web service would be much more complex.

In summary if this a simple change we have an easy UX performance win.