Skip to content

MINOR: Convert Stream-StreamJoin Integration Test to TTD - #7752

Merged
bbejeck merged 2 commits into
apache:trunkfrom
bbejeck:MINOR_convert_streams_joins_tests_to_TTD
Nov 27, 2019
Merged

MINOR: Convert Stream-StreamJoin Integration Test to TTD#7752
bbejeck merged 2 commits into
apache:trunkfrom
bbejeck:MINOR_convert_streams_joins_tests_to_TTD

Conversation

@bbejeck

@bbejeck bbejeck commented Nov 27, 2019

Copy link
Copy Markdown
Member

Convert StreamStreamJoinIntegrationTest to TTD for more stable testing.

Committer Checklist (excluded from commit message)

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

@bbejeck bbejeck added streams tests Test fixes (including flaky tests) labels Nov 27, 2019
@bbejeck

bbejeck commented Nov 27, 2019

Copy link
Copy Markdown
Member Author

ping 1 of @mjsax, @vvcephei, @cadonna, and @ableegoldman for review

@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.

LGTM (assuming Jenkins passes). Feel free to merge when build is green.

@bbejeck

bbejeck commented Nov 27, 2019

Copy link
Copy Markdown
Member Author

Java 8 and Java 11/2.13 passed
Java 11/2.12 timed out Build timed out (after 270 minutes). Marking the build as aborted.

retest this please

@bbejeck
bbejeck merged commit a50b7c4 into apache:trunk Nov 27, 2019
@bbejeck

bbejeck commented Nov 27, 2019

Copy link
Copy Markdown
Member Author

Merged #7752 into trunk.

bbejeck added a commit that referenced this pull request Dec 4, 2019
Convert StreamStreamJoinIntegrationTest to TTD for more stable testing.

Reviewers: Matthias J. Sax <mjsax@apache.org>
@bbejeck

bbejeck commented Dec 5, 2019

Copy link
Copy Markdown
Member Author

cherry-picked to 2.4

@bbejeck
bbejeck deleted the MINOR_convert_streams_joins_tests_to_TTD branch July 10, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

streams tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants