pinot icon indicating copy to clipboard operation
pinot copied to clipboard

Reduce load time for Pinot UI Home Page

Open deepthi912 opened this issue 1 year ago • 4 comments

In the Pinot UI home page currently the following is the behaviour: Redundant calls Multiple /instances calls, 2 for each component type (total 4*2=8) although one call should provide information for all Multiple /zk/ls?path=%2Fpinot%2FLIVEINSTANCES calls, 1 each component type (total 4) although one call should provide information for all.

We can try to reduce these redundant calls: Reuse the response from /instances to display instance counts and instance info tabs Reuse the response from /zk/ls?path=%2Fpinot%2FLIVEINSTANCES to display instance status in instance info tabs.

Left over task from : https://github.com/apache/pinot/issues/13278

deepthi912 avatar Sep 23 '24 22:09 deepthi912

@deepthi912 is it just on the homepage or do we have to address redundant api calls from other pages in the UI as well

jayeshchoudhary avatar Sep 24 '24 17:09 jayeshchoudhary

Not necessarily with Home page, if there are redundant calls from these apis made in other pages, we can change that as well. Does that answer the question?

deepthi912 avatar Sep 24 '24 17:09 deepthi912

okay, will try to check each page and raise pr one page at a time. Starting with home page.

jayeshchoudhary avatar Sep 24 '24 17:09 jayeshchoudhary

@jayeshchoudhary Does #14295 resolve this issue completely? Or are you going to make more improvements?

Jackie-Jiang avatar Oct 24 '24 17:10 Jackie-Jiang

I think this issue can be closed now.

deepthi912 avatar Oct 28 '24 21:10 deepthi912