Skip to content

MINOR: Rename unit test class for MeteredTimestampedWindowStore#6826

Merged
bbejeck merged 2 commits into
apache:trunkfrom
cadonna:rename_test_class
May 29, 2019
Merged

MINOR: Rename unit test class for MeteredTimestampedWindowStore#6826
bbejeck merged 2 commits into
apache:trunkfrom
cadonna:rename_test_class

Conversation

@cadonna

@cadonna cadonna commented May 27, 2019

Copy link
Copy Markdown
Member

Committer Checklist (excluded from commit message)

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

@cadonna

cadonna commented May 27, 2019

Copy link
Copy Markdown
Member Author

@bbejeck

bbejeck commented May 28, 2019

Copy link
Copy Markdown
Member

Failures unrelated

retest this please

@bbejeck

bbejeck commented May 28, 2019

Copy link
Copy Markdown
Member

Java 11 build failed Error cloning remote repo 'origin' Java 8 failed with

kafka.integration.UncleanLeaderElectionTest.testUncleanLeaderElectionDisabled
kafka.integration.UncleanLeaderElectionTest.testUncleanLeaderElectionDisabledByTopicOverride

retest this please

@guozhangwang

Copy link
Copy Markdown
Contributor

@cadonna seems like you need to rebase now..

@cadonna

cadonna commented May 29, 2019

Copy link
Copy Markdown
Member Author

Failure unrelated
Retest this, please.

@bbejeck
bbejeck merged commit e7da042 into apache:trunk May 29, 2019
@bbejeck

bbejeck commented May 29, 2019

Copy link
Copy Markdown
Member

Merged #6826 into trunk

bbejeck pushed a commit that referenced this pull request May 29, 2019
)

Reviewers: Guozhang Wang <wangguoz@gmail.com>,  Bill Bejeck <bbejeck@gmail.com>
@bbejeck

bbejeck commented May 29, 2019

Copy link
Copy Markdown
Member

cherry-picked #6826 to 2.3

pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
…ache#6826)

Reviewers: Guozhang Wang <wangguoz@gmail.com>,  Bill Bejeck <bbejeck@gmail.com>
@cadonna
cadonna deleted the rename_test_class branch October 21, 2019 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants