New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KAFKA-16139: Fix StreamsUpgradeTest #15199
KAFKA-16139: Fix StreamsUpgradeTest #15199
Conversation
Adds version 3.5 to the possible values for config upgrade_from.
@mjsax Could you have a look since you reviewed also the PR that added upgrade tests to version 3.5? |
I started Streams system tests here: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/6034/ |
Is this fix sufficient? Don't we not also need to add |
I just checked the test failures on #15151 (cf last comment) and it failed because |
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Merged to |
Follow-up PR to add |
Follow up to apache#15199 Remove unnecessary import and add new member variable.
@@ -71,6 +71,7 @@ | |||
import static org.apache.kafka.common.config.ConfigDef.Range.between; | |||
import static org.apache.kafka.common.config.ConfigDef.ValidString.in; | |||
import static org.apache.kafka.common.config.ConfigDef.parseType; | |||
import static org.apache.kafka.streams.internals.UpgradeFromValues.UPGRADE_FROM_35; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After merging and doing the follow up PR to add 3.6
, I realized that using this import is not what we want to do.
Call for review (follow up cleanup PR): #15208
Follow up to apache#15199 Remove unnecessary import and add new member variable.
Follow up to #15199 Remove unnecessary import and add new member variable. Reviewers: Walker Carlson <wcarlson@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from. Reviewers: Matthias J. Sax <matthias@confluent.io>
Adds version 3.5 to the possible values for config upgrade_from.
Committer Checklist (excluded from commit message)