jadami10 opened a new pull request, #12210:
URL: https://github.com/apache/pinot/pull/12210

   This is a UI feature change addressing #10504. Apologies for the humungous 
PR, but I figured it would be best to knock this out in 1 go. There's a couple 
of things happening here:
   - added a small "NotFound" page if you navigate to 
tables/schemas/instances/segments that don't exist. It doesn't happen often, 
but previous the UI just broke if you did this
   - as much as I could, all UI elements should now load async. Meaning the UI 
will immediately render with "loading" indicators, and then the data will 
populate the page
     - for a small cluster, this change is likely imperceptible
     - for a large cluster, everything will feel a lot snappier. Table sizes, 
table status, and instance counts are the slowest things to load. Before this, 
they would hold up rendering the entire page
     - restarting a server would also previously induce a lot of slowness in 
the UI. The UI wouldn't load until the /size endpoint from controller->server 
timed out. Now it just shows that it's "loading" but the rest of the data is on 
the page
   - I ran prettier (with the Pinot config) over each file I touched. This was 
mostly because writing JS without a linter is hard, and it was simpler to just 
reformat the whole file rather than try to get my changes correct.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to