[GitHub] [pinot] jadami10 commented on pull request #9741: render the tables list even as the table sizes are loading

[GitBox <gi...@apache.org>]

jadami10 commented on PR #9741:
URL: https://github.com/apache/pinot/pull/9741#issuecomment-1304853026

   @Jackie-Jiang, took me quite a while to get back to to this. But pulling your old comment here
   
   > This is a great feature!
   > 
   > Let's list down all the data need to be fetched, and we may decide which ones should be loaded sync and which ones async.
   > Also, I'm not sure if we want to query the table size on the top level page. In an environment with hundreds or even more tables, that will create quite some traffic to the servers. We should limit the details to the table level page when there are too many tables (if not already done that way).
   > 
   > If we order the cost of operations from least expensive to most expensive:
   > 
   > List tables / schemas
   > Read schema to get column stats
   > Read table IS and EV to get segment availability
   > Read table size from ser
   
   I think this PR I wanted to limit this to just the async change and not actually change any of the data loaded here.


-- 
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: commits-unsubscribe@pinot.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org