Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Doc fixes: Fix format and other small errors in config documentation #13661

Merged
merged 9 commits into from Jul 10, 2023
Merged

Doc fixes: Fix format and other small errors in config documentation #13661

merged 9 commits into from Jul 10, 2023
+61 −63

Conversation

cherylws
Copy link
Contributor

@cherylws cherylws commented May 2, 2023 •
edited

  • Various formatting fixes in the config docs.

@bbejeck
Screenshots of built fixes:

control.plane.listener.name:
image

controller.quorum.voters:
image

partitioner.class:
image

delete.retention.ms:
image

@cherylws cherylws marked this pull request as ready for review May 26, 2023 16:52
@cherylws cherylws changed the title Fix format and other small errors in config documentation Doc fixes: Fix format and other small errors in config documentation May 26, 2023
@bbejeck bbejeck added the docs label Jul 10, 2023
@bbejeck
Copy link
Contributor

bbejeck commented Jul 10, 2023

Failures unrelated

JDK 8 and Scala 2.12 / testUnregisterBroker() – org.apache.kafka.controller.QuorumControllerTest
JDK 8 and Scala 2.12 / testConfigurationOperations() – org.apache.kafka.controller.QuorumControllerTest
JDK 17 and Scala 2.13 / testMultiNodeCluster() – org.apache.kafka.connect.mirror.integration.DedicatedMirrorIntegrationTest
JDK 17 and Scala 2.13 / testConnectorBoundary – org.apache.kafka.connect.integration.ExactlyOnceSourceIntegrationTest
JDK 11 and Scala 2.13 / testOutdatedCoordinatorAssignment() – org.apache.kafka.clients.consumer.internals.CooperativeConsumerCoordinatorTest

@bbejeck bbejeck merged commit e985087 into apache:trunk Jul 10, 2023
1 check failed
@bbejeck
Copy link
Contributor

bbejeck commented Jul 10, 2023

Merged #13661 into trunk

Cerchie pushed a commit to Cerchie/kafka that referenced this pull request Jul 25, 2023
@cherylws
Doc fixes: Fix format and other small errors in config documentation (a…
e28c0a0 
…pache#13661)

Various formatting fixes in the config docs.

Reviewers: Bill Bejeck <bbejeck@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bbejeck bbejeck bbejeck approved these changes

Assignees
No one assigned
Labels
docs
Projects
None yet
Milestone
No milestone
2 participants
@cherylws @bbejeck