[GitHub] [incubator-pinot] mayankshriv commented on pull request #7167: Bump up data table version to V4 to optimize space usage.

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

mayankshriv commented on pull request #7167:
URL: https://github.com/apache/incubator-pinot/pull/7167#issuecomment-881080524


   > since presto connector will access pinot server with segment query and wrap up DataTable by itself. Is it possible to make the query itself contain the version of which DataTable protocol to use?
   > So the Pinot server can work with v3 broker/v4 broker or v2 presto.
   
   @mqliang I have the same concern as @xiangfu0. We already ran into this issue when you did the v2 to v3 upgrade. Even though this might be thought of as internal Pinot data structure, there are connectors that use this data table, making it an externally exposed data structure, and we need to be careful with incrementing version numbers in backward incompatible way.


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