[GitHub] [incubator-pinot] mcvsubbu edited a comment on pull request #7142: Added a query param to /tables for getting sorted table names based on time metadata

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

mcvsubbu edited a comment on pull request #7142:
URL: https://github.com/apache/incubator-pinot/pull/7142#issuecomment-877831082


   Why should sorting be the function of the API? There are external tools available to sort the results and parse them in various ways. If we need to sort by retention and then by table name, would you then extend the API? I am not in favor of adding more complexity to the API when there is more flexibility doing such operations outside of Pinot.


-- 
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