Skip to content

Conversation

@brkyvz
Copy link
Contributor

@brkyvz brkyvz commented Dec 15, 2016

What changes were proposed in this pull request?

Use recentProgress instead of lastProgress and filter out last non-zero value. Also add eventually to the latest assertQuery similar to first assertQuery

How was this patch tested?

Ran test 1000 times

@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70163 has finished for PR 16287 at commit cedaafd.

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

@brkyvz brkyvz changed the title [SPARK-18868][FLAKY-TEST] Deflake StreamingQueryListenerSuite: single listener, check trigger... [SPARK-18868][FLAKY-TEST][DO-NOT-MERGE] Deflake StreamingQueryListenerSuite: single listener, check trigger... Dec 15, 2016
@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70202 has finished for PR 16287 at commit 32d1ca8.

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

@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70207 has finished for PR 16287 at commit 50d1e44.

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

@brkyvz
Copy link
Contributor Author

brkyvz commented Dec 15, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70203 has finished for PR 16287 at commit d4bbb45.

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

@brkyvz
Copy link
Contributor Author

brkyvz commented Dec 15, 2016

@zsxwing @tdas This passed 1,000 times on 2 jenkins tests. The latest test started tipping over after 1,100 retries

@brkyvz brkyvz changed the title [SPARK-18868][FLAKY-TEST][DO-NOT-MERGE] Deflake StreamingQueryListenerSuite: single listener, check trigger... [SPARK-18868][FLAKY-TEST] Deflake StreamingQueryListenerSuite: single listener, check trigger... Dec 15, 2016
@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70211 has finished for PR 16287 at commit fc2bc34.

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

@zsxwing
Copy link
Member

zsxwing commented Dec 15, 2016

LGTM. Merging to master and 2.1.

@asfgit asfgit closed this in 9c7f83b Dec 15, 2016
asfgit pushed a commit that referenced this pull request Dec 15, 2016
… listener, check trigger...

## What changes were proposed in this pull request?

Use `recentProgress` instead of `lastProgress` and filter out last non-zero value. Also add eventually to the latest assertQuery similar to first `assertQuery`

## How was this patch tested?

Ran test 1000 times

Author: Burak Yavuz <[email protected]>

Closes #16287 from brkyvz/SPARK-18868.

(cherry picked from commit 9c7f83b)
Signed-off-by: Shixiong Zhu <[email protected]>
@SparkQA
Copy link

SparkQA commented Dec 15, 2016

Test build #70212 has finished for PR 16287 at commit 95824f3.

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

@SparkQA
Copy link

SparkQA commented Dec 16, 2016

Test build #70208 has finished for PR 16287 at commit 50d1e44.

  • This patch fails from timeout after a configured wait of `250m`.
  • This patch merges cleanly.
  • This patch adds no public classes.

uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
… listener, check trigger...

## What changes were proposed in this pull request?

Use `recentProgress` instead of `lastProgress` and filter out last non-zero value. Also add eventually to the latest assertQuery similar to first `assertQuery`

## How was this patch tested?

Ran test 1000 times

Author: Burak Yavuz <[email protected]>

Closes apache#16287 from brkyvz/SPARK-18868.
@brkyvz brkyvz deleted the SPARK-18868 branch February 3, 2019 20:58
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.

3 participants