saurabhd336 opened a new pull request, #11576:
URL: https://github.com/apache/pinot/pull/11576
This API is primarily for the controller UI -> tables -> <table name> ->
reload status -> STATUS ui element. Right now it uses the
`/segments/{tableName}/metadata?type={type}&columns=*` API to display the
column level index details. This has a few problems
1) This is an expensive API call especially for large tables. This leads to
bad ux when using the reload status UI feature
2) The UI logic to build tableLevel column -> index mapping using this
segmentName level metadata is simply wrong (uss the first segment's metadata to
build the view).
3) Since Index-spi changes are merged, this API does not return metadata of
dynamically added index types, but only some hardcoded ones.
This new API's reponse looks like this
```
{
"totalSegments": 31,
"columnToIndexesCount":
{
"FlightNum":
{
"dictionary": 31,
"bloom": 0,
"null": 0,
"forward": 31,
"fst": 0,
"json": 0,
"range": 0,
"h3": 0,
"text": 0,
"inverted": 0,
"some-dynamically-injected-index-type": 31,
},
"Origin":
{
"dictionary": 31,
"bloom": 0,
"null": 0,
"forward": 31,
"fst": 0,
"json": 0,
"range": 0,
"h3": 0,
"text": 0,
"inverted": 0,
"some-dynamically-injected-index-type": 0,
}
...
}
```
I'll followup with @jayeshchoudhary to move the UI API call to this new one,
and how best to display the UI element once this PR is merged.
--
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-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
