Skip to content

Deprecate older data tables#11140

Closed
saurabhd336 wants to merge 2 commits intoapache:masterfrom
saurabhd336:dataTableDeprecate
Closed

Deprecate older data tables#11140
saurabhd336 wants to merge 2 commits intoapache:masterfrom
saurabhd336:dataTableDeprecate

Conversation

@saurabhd336
Copy link
Contributor

Deprecate and cleanup code related to older data table versions

@Jackie-Jiang
Copy link
Contributor

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.

@tibrewalpratik17
Copy link
Contributor

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.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants