Skip to content

Conversation

@MaxGekk
Copy link
Member

@MaxGekk MaxGekk commented Apr 24, 2020

What changes were proposed in this pull request?

Retry tests from CliSuite. This is temporary changes till someone stabilises the test suite.

Why are the changes needed?

To not disturb other PRs.

Does this PR introduce any user-facing change?

No

How was this patch tested?

By running the test suite locally via:

./build/sbt -Phive -Phive-thriftserver "test:testOnly *CliSuite"

@MaxGekk
Copy link
Member Author

MaxGekk commented Apr 24, 2020

@cloud-fan @gatorsmile @HyukjinKwon Please, take a look at it.

@SparkQA
Copy link

SparkQA commented Apr 24, 2020

Test build #121769 has finished for PR 28329 at commit 99099e5.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HeartSaVioR
Copy link
Contributor

What about trying below PRs first and see whether the help, and revisit this?

#28156
#28055

@MaxGekk
Copy link
Member Author

MaxGekk commented Apr 25, 2020

@HeartSaVioR This is a temporary fix till the issue of test flakiness is resolved by PRs you mentioned or by some other PRs. /cc @cloud-fan @dongjoon-hyun @srowen @gatorsmile

@MaxGekk
Copy link
Member Author

MaxGekk commented Apr 30, 2020

It seems @juliuszsompolski 's PR #28156 fixed the issue. I am closing this.

@MaxGekk MaxGekk closed this Apr 30, 2020
@MaxGekk MaxGekk deleted the retry-CliSuite branch June 5, 2020 19:48
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