Skip to content

KAFKA-8147: Add changelog topic configuration to KTable suppress#8965

Merged
mjsax merged 4 commits into
apache:trunkfrom
highluck:suppress-config-docs
Dec 4, 2020
Merged

KAFKA-8147: Add changelog topic configuration to KTable suppress#8965
mjsax merged 4 commits into
apache:trunkfrom
highluck:suppress-config-docs

Conversation

@highluck

@highluck highluck commented Jul 1, 2020

Copy link
Copy Markdown
Contributor

KAFKA-8147: Add changelog topic configuration to KTable suppress

Committer Checklist (excluded from commit message)

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

@highluck

highluck commented Jul 1, 2020

Copy link
Copy Markdown
Contributor Author

@mjsax
Should I do it like this way?
thank you!

@mjsax mjsax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the PR @highluck! Just a few minor comments.

Comment thread docs/streams/upgrade-guide.html Outdated
Comment thread docs/streams/upgrade-guide.html Outdated
<dt><code>withLoggingDisabled()</code></dt>
<dd>
This configures the suppression operator to disable logging for changelog entries.
</dd>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not sure if we want to add this in this example? \cc @vvcephei WDYT?

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.

Thanks @mjsax , I agree; this is an advanced configuration that shouldn't be used most of the time, so I wouldn't put it in the example.

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.

update code thanks!

@highluck

highluck commented Jul 8, 2020

Copy link
Copy Markdown
Contributor Author

@mjsax
thank you for review!
updated code!

@highluck
highluck requested a review from mjsax November 11, 2020 11:40

@mjsax mjsax left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for updating the PR @highluck

Comment thread docs/streams/upgrade-guide.html Outdated
Comment thread docs/streams/upgrade-guide.html Outdated
@highluck
highluck requested a review from mjsax November 22, 2020 15:34
@mjsax
mjsax merged commit 88c8180 into apache:trunk Dec 4, 2020
mjsax pushed a commit that referenced this pull request Dec 4, 2020
Reviewer: Matthias J. Sax <matthias@confluent.io>, John Roesler <john@confluent.io>
mjsax pushed a commit that referenced this pull request Dec 4, 2020
Reviewer: Matthias J. Sax <matthias@confluent.io>, John Roesler <john@confluent.io>
@mjsax

mjsax commented Dec 4, 2020

Copy link
Copy Markdown
Member

Merged to trunk and cherry-picked to 2.7 and 2.6 branches.

@mjsax

mjsax commented Dec 4, 2020

Copy link
Copy Markdown
Member

Thanks a lot for the PR @highluck!

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