Skip to content

KAFKA-7223: Part 3 preview#5688

Closed
vvcephei wants to merge 10 commits into
apache:trunkfrom
vvcephei:KAFKA-7223-in-mem-suppress-code-only
Closed

KAFKA-7223: Part 3 preview#5688
vvcephei wants to merge 10 commits into
apache:trunkfrom
vvcephei:KAFKA-7223-in-mem-suppress-code-only

Conversation

@vvcephei

Copy link
Copy Markdown
Contributor

"preview" of Part 3 of suppression.

This PR is provided to aid understanding of #5687.

To keep the diff legible, I dropped all the test changes (so the tests will actually not pass on this PR).

After #5687 is merged, I will revamp this one to become Part 3

Committer Checklist (excluded from commit message)

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

@vvcephei

Copy link
Copy Markdown
Contributor Author

I've rebased on #5687 to pull in some logic changes that were a consequence of test feedback.
Reminder: this code won't compile until #5687 is merged.

@vvcephei

Copy link
Copy Markdown
Contributor Author

Discarding this "preview" PR in favor of a new Part 3 PR...

@vvcephei vvcephei closed this Sep 25, 2018
@vvcephei

Copy link
Copy Markdown
Contributor Author

This PR was superseded by #5693

@vvcephei
vvcephei deleted the KAFKA-7223-in-mem-suppress-code-only branch September 25, 2018 20:58
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.

1 participant