Skip to content

KAFKA-9739: 2.3 null child node fix#8419

Merged
bbejeck merged 2 commits intoapache:2.3from
bbejeck:KAFKA_9739_23_null_child_node_fix
Apr 4, 2020
Merged

KAFKA-9739: 2.3 null child node fix#8419
bbejeck merged 2 commits intoapache:2.3from
bbejeck:KAFKA_9739_23_null_child_node_fix

Conversation

@bbejeck
Copy link
Copy Markdown
Member

@bbejeck bbejeck commented Apr 3, 2020

A port of #8400 for 2.3. The process of sorting source and sink nodes changed in 2.4, so we can't cherry-pick the PR directly as we need to update the expected topology to what it would be in the 2.3 version.

Committer Checklist (excluded from commit message)

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

bbejeck added 2 commits April 3, 2020 15:35
2.4 port of apache#8400 since cherry-picking not possible

Reviewers: John Roesler <john@confluent.io>
…sort source nodes so to merge this PR with 2.3 I had to update the expected topology to what was expected on 2.3
Copy link
Copy Markdown
Contributor

@vvcephei vvcephei left a comment

Choose a reason for hiding this comment

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

Thanks again, again!

@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Apr 3, 2020

retest this please

@bbejeck bbejeck merged commit 477d43a into apache:2.3 Apr 4, 2020
@bbejeck bbejeck deleted the KAFKA_9739_23_null_child_node_fix branch April 4, 2020 15:46
@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Apr 4, 2020

Merged #8419 into 2.3

bbejeck added a commit that referenced this pull request Apr 4, 2020
A port of #8400 for 2.3. The process of sorting source and sink nodes changed in 2.4, so we can't cherry-pick the PR directly as we need to update the expected topology to what it would be in the 2.3 version.

Reviewers: John Roesler <john@confluent.io>, Andrew Choi <a24choi@edu.uwaterloo.ca>
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