[GitHub] [incubator-pinot] mqliang edited a comment on pull request #7167: Bump up data table version to V4 to optimize space usage.

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

mqliang edited a comment on pull request #7167:
URL: https://github.com/apache/incubator-pinot/pull/7167#issuecomment-881080649


   > 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.
   
   @xiangfu0 @mayankshriv
   We will eventually stop supporting V2 and V3, so it's better to update codebase of presto connector to follow up the most recent pinot. But I do think it a good idea to to allow client specify DataTable version, this will make client happy: client only need make minimal change after pinot is upgraded( by specifying a old DataTable version), this will give client developer more time to upgrade their code. Let's pull @mcvsubbu @siddharthteotia to chime in.


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