Skip to content

Docs: Include ZK Deprecation notice and information#14031

Merged
mjsax merged 2 commits intoapache:trunkfrom
forlack:patch-1
Jul 18, 2023
Merged

Docs: Include ZK Deprecation notice and information#14031
mjsax merged 2 commits intoapache:trunkfrom
forlack:patch-1

Conversation

@forlack
Copy link
Copy Markdown
Contributor

@forlack forlack commented Jul 17, 2023

Include information on ZK deprecation in documentation and provide timelines for removal. With AK 3.5 ZK is marked deprecated for new clusters. This section aims to provide information on what deprecation means, timelines, and path to migration with KRaft.

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Include information on ZK deprecation in documentation and provide timelines for removal.
Copy link
Copy Markdown
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

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

Thanks

Copy link
Copy Markdown
Member

@divijvaidya divijvaidya left a comment

Choose a reason for hiding this comment

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

Thank you for making these changes. Looks good to me.

Comment thread docs/ops.html Outdated
Added suggestion form AK PR
@forlack
Copy link
Copy Markdown
Contributor Author

forlack commented Jul 18, 2023

I updated the sentence suggested by @ijuma

@forlack forlack requested a review from ijuma July 18, 2023 20:29
@mjsax mjsax merged commit a82df30 into apache:trunk Jul 18, 2023
mjsax pushed a commit that referenced this pull request Jul 18, 2023
…14031)

Reviewers: Ismael Juma <ismael@confluent.io>, Mickael Maison <mickael.maison@gmail.com>, Divij Vaidya <diviv@amazon.com>, Matthias J. Sax <matthias@confluent.io>
@mjsax
Copy link
Copy Markdown
Member

mjsax commented Jul 18, 2023

Merged to trunk and cherry-picked to 3.5 branch. Will take care to get the changes into kafka-site, too.

jolshan added a commit to confluentinc/kafka that referenced this pull request Jul 18, 2023
* ak/trunk: (110 commits)
  MINOR: Update docs to include ZK deprecation notice and information (apache#14031)
  KAFKA-15091: Fix misleading Javadoc for SourceTask::commit (apache#13948)
  KAFKA-14669: Use the generated docs for MirrorMaker configs in the doc (apache#13658)
  KAFKA-14953: Add tiered storage related metrics (apache#13944)
  KAFKA-15121: Implement the alterOffsets method in the FileStreamSourceConnector and the FileStreamSinkConnector (apache#13945)
  Revert "MINOR: Update .asf.yaml file with refreshed github_whitelist, and collaborators" (apache#14037)
  MINOR: Update .asf.yaml file with refreshed github_whitelist, and collaborators
  KAFKA-14737: Move kafka.utils.json to server-common (apache#13585)
  KAFKA-14647: Move TopicFilter to server-common/utils (apache#13158)
  MINOR: remove unused variable in examples (apache#14021)
  ...
jeqo pushed a commit to jeqo/kafka that referenced this pull request Jul 21, 2023
…pache#14031)

Reviewers: Ismael Juma <ismael@confluent.io>, Mickael Maison <mickael.maison@gmail.com>, Divij Vaidya <diviv@amazon.com>, Matthias J. Sax <matthias@confluent.io>
Cerchie pushed a commit to Cerchie/kafka that referenced this pull request Jul 25, 2023
…pache#14031)

Reviewers: Ismael Juma <ismael@confluent.io>, Mickael Maison <mickael.maison@gmail.com>, Divij Vaidya <diviv@amazon.com>, Matthias J. Sax <matthias@confluent.io>
jeqo pushed a commit to aiven/kafka that referenced this pull request Aug 15, 2023
…pache#14031)

Reviewers: Ismael Juma <ismael@confluent.io>, Mickael Maison <mickael.maison@gmail.com>, Divij Vaidya <diviv@amazon.com>, Matthias J. Sax <matthias@confluent.io>
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.

5 participants