You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by "saurabhd336 (via GitHub)" <gi...@apache.org> on 2023/07/20 08:31:10 UTC
[GitHub] [pinot] saurabhd336 opened a new pull request, #11140: Deprecate older data tables
saurabhd336 opened a new pull request, #11140:
URL: https://github.com/apache/pinot/pull/11140
Deprecate and cleanup code related to older data table versions
--
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
[GitHub] [pinot] tibrewalpratik17 commented on pull request #11140: Deprecate older data tables
Posted by "tibrewalpratik17 (via GitHub)" <gi...@apache.org>.
tibrewalpratik17 commented on PR #11140:
URL: https://github.com/apache/pinot/pull/11140#issuecomment-1646328404
> We bumped the default data table version in https://github.com/apache/pinot/pull/10543, which is not released yet. I don't think we can remove the support for old data tables right now because that can cause the system not able to upgrade. If we are running new broker and old server, broker won't understand the data table sent by the server, and our upgrade sequence is controller -> broker -> server.
+1 we recently found out that we are running on an older data table version and started upgrading the version. Removing support for older data table versions will be a major backward incompatibility change.
--
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
[GitHub] [pinot] Jackie-Jiang commented on pull request #11140: Deprecate older data tables
Posted by "Jackie-Jiang (via GitHub)" <gi...@apache.org>.
Jackie-Jiang commented on PR #11140:
URL: https://github.com/apache/pinot/pull/11140#issuecomment-1646324994
We bumped the default data table version in #10543, which is not released yet. I don't think we can remove the support for old data tables right now because that can cause the system not able to upgrade. If we are running new broker and old server, broker won't understand the data table sent by the server, and our upgrade sequence is controller -> broker -> server.
--
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