Skip to content

[DO-NOT-MERGE][TEST] Investigate StateStoreSuite flakiness#12160

Closed
tdas wants to merge 3 commits intoapache:masterfrom
tdas:state-store-flaky-fix
Closed

[DO-NOT-MERGE][TEST] Investigate StateStoreSuite flakiness#12160
tdas wants to merge 3 commits intoapache:masterfrom
tdas:state-store-flaky-fix

Conversation

@tdas
Copy link
Contributor

@tdas tdas commented Apr 4, 2016

No description provided.

@tdas tdas changed the title [DO-NOT-MERGE][TEST] Investigate StateStoreSuite flakiness [DO-NOT-MERGE][TEST] Investigate StateStoreSuite and ContinuousQueryManagerSuiteflakiness Apr 5, 2016
@tdas tdas changed the title [DO-NOT-MERGE][TEST] Investigate StateStoreSuite and ContinuousQueryManagerSuiteflakiness [DO-NOT-MERGE][TEST] Investigate StateStoreSuite and ContinuousQueryManagerSuite flakiness Apr 5, 2016
@tdas tdas force-pushed the state-store-flaky-fix branch from 91bc8e0 to db0a4da Compare April 5, 2016 00:47
@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #54909 has finished for PR 12160 at commit 80456d6.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2746 has finished for PR 12160 at commit 80456d6.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2745 has finished for PR 12160 at commit 80456d6.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2747 has finished for PR 12160 at commit 80456d6.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2749 has finished for PR 12160 at commit 80456d6.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #54916 has finished for PR 12160 at commit db0a4da.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #54915 has finished for PR 12160 at commit 91bc8e0.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2754 has finished for PR 12160 at commit db0a4da.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2753 has finished for PR 12160 at commit db0a4da.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2755 has finished for PR 12160 at commit db0a4da.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2748 has finished for PR 12160 at commit 80456d6.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2751 has finished for PR 12160 at commit db0a4da.

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

@SparkQA
Copy link

SparkQA commented Apr 5, 2016

Test build #2752 has finished for PR 12160 at commit db0a4da.

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

@tdas tdas changed the title [DO-NOT-MERGE][TEST] Investigate StateStoreSuite and ContinuousQueryManagerSuite flakiness [DO-NOT-MERGE][TEST] Investigate StateStoreSuite flakiness Apr 7, 2016
@tdas tdas force-pushed the state-store-flaky-fix branch from db0a4da to 6ce7e6a Compare April 19, 2016 00:48
@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2820 has finished for PR 12160 at commit 6ce7e6a.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2819 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2821 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2822 has finished for PR 12160 at commit 6ce7e6a.

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

@tdas
Copy link
Contributor Author

tdas commented Apr 19, 2016

@zsxwing Can you take a look.
I havent quite nailed down why this was flaky, but my best guess is that its something to do with StreamingAggregationSuite leaving StateStore background thread running. Now this can happen in any test suite and it should not matter. So for to prevent this suite from failing i am making sure StoreStore is stopped before running this suite. Does this sound good.

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #56162 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2823 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2826 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2825 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 19, 2016

Test build #2827 has finished for PR 12160 at commit 6ce7e6a.

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

@SparkQA
Copy link

SparkQA commented Apr 20, 2016

Test build #56402 has finished for PR 12160 at commit a361ea7.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Apr 21, 2016

Test build #56441 has finished for PR 12160 at commit 1d2fb46.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@tdas tdas closed this May 4, 2016
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.

2 participants