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.

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

KAFKA-7990: Close streams at the end in KafkaStreamsTest #6334

Merged
merged 1 commit into from Feb 27, 2019

Conversation

guozhangwang
Copy link
Contributor

This fix is already in 2.1+ branches, but did not get into older branches.

Should be cherry-picked all the way to 0.10.2.

Committer Checklist (excluded from commit message)

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

@guozhangwang
Copy link
Contributor Author

@mjsax @bbejeck for reviews.

@mjsax
Copy link
Member

mjsax commented Feb 27, 2019

@guozhangwang Should we cherry-pick the old fix instead? Or is this not possible?

@mjsax mjsax added the streams label Feb 27, 2019
@guozhangwang
Copy link
Contributor Author

The original PR (5618) is a very big one that covers lots of files :) Many of them are either covered already or not added in older branches though.

@mjsax
Copy link
Member

mjsax commented Feb 27, 2019

Thanks. LGTM.

@guozhangwang guozhangwang merged commit 1158c74 into apache:2.0 Feb 27, 2019
@guozhangwang guozhangwang deleted the K7990-fix-test branch February 27, 2019 06:28
guozhangwang added a commit that referenced this pull request Feb 27, 2019
This fix is already in 2.1+ branches, but did not get into older branches.

Should be cherry-picked all the way to 0.10.2.

Reviewers: Matthias J. Sax <mjsax@apache.org>
guozhangwang added a commit that referenced this pull request Feb 27, 2019
This fix is already in 2.1+ branches, but did not get into older branches.

Should be cherry-picked all the way to 0.10.2.

Reviewers: Matthias J. Sax <mjsax@apache.org>
guozhangwang added a commit that referenced this pull request Feb 27, 2019
This fix is already in 2.1+ branches, but did not get into older branches.

Should be cherry-picked all the way to 0.10.2.

Reviewers: Matthias J. Sax <mjsax@apache.org>
guozhangwang added a commit that referenced this pull request Feb 27, 2019
This fix is already in 2.1+ branches, but did not get into older branches.

Should be cherry-picked all the way to 0.10.2.

Reviewers: Matthias J. Sax <mjsax@apache.org>
@guozhangwang
Copy link
Contributor Author

Cherry-picked to 1.1/1.0/0.11.0/0.10.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants