Skip to content

MINOR: Update dynamic broker configuration doc for truststore update#4954

Merged
hachikuji merged 1 commit intoapache:trunkfrom
rajinisivaram:MINOR-dynamic-config-doc
May 10, 2018
Merged

MINOR: Update dynamic broker configuration doc for truststore update#4954
hachikuji merged 1 commit intoapache:trunkfrom
rajinisivaram:MINOR-dynamic-config-doc

Conversation

@rajinisivaram
Copy link
Copy Markdown
Contributor

Committer Checklist (excluded from commit message)

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

@rajinisivaram rajinisivaram requested a review from hachikuji May 2, 2018 11:30
@omkreddy
Copy link
Copy Markdown
Contributor

omkreddy commented May 7, 2018

LGTM

Comment thread docs/configuration.html
<li><code>log.message.timestamp.difference.max.ms</code></li>
</ul>

From Kafka version 2.0.0 onwards, unclean leader election is automatically enabled by the controller when the config
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe we could leave this out of this PR and @omkreddy can include it in the patch for KAFKA-6835?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@hachikuji The changes to the controller for dynamic broker config update was made under https://issues.apache.org/jira/browse/KAFKA-6526 (already merged). So I thought it will be good to update the broker config doc for default topic configuration. @omkreddy is updating the controller for per-topic-override of unclean.leader.election.enable and I think his patch includes changes to the upgrade docs.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah, ok. I missed the difference between KAFKA-6835 and KAFKA-6526.

Copy link
Copy Markdown
Contributor

@hachikuji hachikuji left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the patch!

@hachikuji hachikuji merged commit 830ee16 into apache:trunk May 10, 2018
ijuma added a commit to ijuma/kafka that referenced this pull request May 11, 2018
…-record-version

* apache-github/trunk:
  KAFKA-6894: Improve err msg when connecting processor with global store (apache#5000)
  KAFKA-6893; Create processors before starting acceptor in SocketServer (apache#4999)
  MINOR: Fix typo in ConsumerRebalanceListener JavaDoc (apache#4996)
  MINOR: Remove deprecated valueTransformer.punctuate (apache#4993)
  MINOR: Update dynamic broker configuration doc for truststore update (apache#4954)
  KAFKA-6870 Concurrency conflicts in SampledStat (apache#4985)
  KAFKA-6361: Fix log divergence between leader and follower after fast leader fail over (apache#4882)
  KAFKA-6813: Remove deprecated APIs in KIP-182, Part II (apache#4976)
  KAFKA-6878 Switch the order of underlying.init and initInternal (apache#4988)
  KAFKA-6299; Fix AdminClient error handling when metadata changes (apache#4295)
  KAFKA-6878: NPE when querying global state store not in READY state (apache#4978)
  KAFKA 6673: Implemented missing override equals method (apache#4745)
  KAFKA-6834: Handle compaction with batches bigger than max.message.bytes (apache#4953)
ying-zheng pushed a commit to ying-zheng/kafka that referenced this pull request Jul 6, 2018
…pache#4954)

Reviewers: Manikumar Reddy O <manikumar.reddy@gmail.com>, Jason Gustafson <jason@confluent.io>
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.

3 participants