Skip to content

MINOR: Update KRaft cluster upgrade documentation for 3.4#13063

Merged
ableegoldman merged 3 commits intoapache:trunkfrom
ableegoldman:3.4-RELEASE-KRaft-upgrade-docs
Jan 9, 2023
Merged

MINOR: Update KRaft cluster upgrade documentation for 3.4#13063
ableegoldman merged 3 commits intoapache:trunkfrom
ableegoldman:3.4-RELEASE-KRaft-upgrade-docs

Conversation

@ableegoldman
Copy link
Member

Followup to add a section for upgrading KRaft clusters to 3.4

Comment on lines 79 to 80
<li>Note that the cluster metadata version cannot be downgraded to a pre-production 3.0.x, 3.1.x, 3.2.x, or 3.3.x version once it has been upgraded.
However, it is possible to downgrade to production versions such as 3.3-IV0, 3.3-IV1, etc.</li>
Copy link
Member Author

@ableegoldman ableegoldman Dec 31, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @mumrah or @cmccabe or @hachikuji or whoever would know best, can you double check this section and specifically this line? The KRaft upgrade section in 3.3 also had 3.3-IV0, 3.3-IV1 as examples but I wasn't sure if it would make sense to bump this to 3.4-IV0, 3.4-IV1, or some combination of those?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pre-production versions are "3.0.x, 3.1.x, 3.2.x". That's before kraft was declared production ready.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotcha, fixed the pre-prod version list. Still not sure what the -IV0 suffixes are, but this make sense now?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this part seems fine now. I had missed something else the first time round and left a comment

</li>
</ol>

<h4><a id="upgrade_3_3_1" href="#upgrade_3_3_1">Upgrading a KRaft-based cluster to 3.3.1 from any version 3.0.x through 3.2.x</a></h4>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also, I noticed the duplicate header -- fixing this won't break anything I hope?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it should be fine to fix this.

Copy link
Member

@ijuma ijuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ableegoldman ableegoldman merged commit 77a395b into apache:trunk Jan 9, 2023
ableegoldman added a commit that referenced this pull request Jan 9, 2023
ijuma added a commit to fvaleri/kafka that referenced this pull request Jan 13, 2023
* apache-github/trunk:
  KAFKA-14601: Improve exception handling in KafkaEventQueue apache#13089
  KAFKA-14367; Add `OffsetCommit` to the new `GroupCoordinator` interface (apache#12886)
  KAFKA-14530: Check state updater more often (apache#13017)
  KAFKA-14304 Use boolean for ZK migrating brokers in RPC/record (apache#13103)
  KAFKA-14003 Kafka Streams JUnit4 to JUnit5 migration part 2 (apache#12301)
  KAFKA-14607: Move Scheduler/KafkaScheduler to server-common (apache#13092)
  KAFKA-14367; Add `OffsetFetch` to the new `GroupCoordinator` interface (apache#12870)
  KAFKA-14557; Lock metadata log dir (apache#13058)
  MINOR: Implement toString method for TopicAssignment and PartitionAssignment (apache#13101)
  KAFKA-12558: Do not prematurely mutate internal partition state in Mirror Maker 2 (apache#11818)
  KAFKA-14540: Fix DataOutputStreamWritable#writeByteBuffer (apache#13032)
  KAFKA-14600: Reduce flakiness in ProducerIdExpirationTest (apache#13087)
  KAFKA-14279: Add 3.3.x streams system tests (apache#13077)
  MINOR: bump streams quickstart pom versions and add to list in gradle.properties (apache#13064)
  MINOR: Update KRaft cluster upgrade documentation for 3.4 (apache#13063)
  KAFKA-14493: Introduce Zk to KRaft migration state machine STUBs in KRaft controller. (apache#12998)
  KAFKA-14570: Fix parenthesis in verifyFullFetchResponsePartitions output (apache#13072)
  MINOR: Remove public mutable fields from ProducerAppendInfo (apache#13091)
guozhangwang pushed a commit to guozhangwang/kafka that referenced this pull request Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants