mayankshriv commented on pull request #7056:
URL: https://github.com/apache/incubator-pinot/pull/7056#issuecomment-860841240
I also recommend the following:
1. We should add other info like the index/dictionary size.
2. Any zk metadata (seems as though you may have already added it).
3. Let's put a bit more thought to the rest apis. For example, I can see it
would be useful to have
- column level
- segment level
- server level
- table level (size of a column index across the table)
- Individual metadata (eg size, dictionary size cardinality etc),
returning giant blog everytime is not a good idea, so we should have a
hierarchical path that allows to zoom in/out. (This can be an enhancement, but
we should layout the api hierarchy from the start).
--
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.
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
