Jackie-Jiang commented on PR #11913: URL: https://github.com/apache/pinot/pull/11913#issuecomment-1787692970
The change itself looks good. My concern is that we are pulling column level index info for every single segments, which can freeze the browser, or even crash the controller for large tables. IMO we don't need to show the index info when checking the reload status, just the segment count should be good enough. We can add a separate button to check the index info, which shows the desired index info from table config, and optionally pull some segments index info from servers (we need to limit segments pulled). Do you know which UI action will pull the column level index info for all segments? We should remove all of them. cc @walterddr I remember you opened an issue for this problem? -- 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]
