Skip to content

Conversation

@brkyvz
Copy link
Contributor

@brkyvz brkyvz commented Dec 2, 2015

We need to make sure that the last entry is indeed the last entry in the queue.

@brkyvz
Copy link
Contributor Author

brkyvz commented Dec 2, 2015

cc @zsxwing

@JoshRosen
Copy link
Contributor

@SparkQA
Copy link

SparkQA commented Dec 2, 2015

Test build #47086 has finished for PR 10110 at commit 0aa4680.

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

@SparkQA
Copy link

SparkQA commented Dec 3, 2015

Test build #47107 has finished for PR 10110 at commit d027fbd.

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

@brkyvz
Copy link
Contributor Author

brkyvz commented Dec 4, 2015

cc @tdas

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks this issue should be a blocker since writeAheadLog batching is enabled by default.

@zsxwing
Copy link
Member

zsxwing commented Dec 7, 2015

LGTM for the patch.

asfgit pushed a commit that referenced this pull request Dec 7, 2015
…y when Jenkins load is high

We need to make sure that the last entry is indeed the last entry in the queue.

Author: Burak Yavuz <[email protected]>

Closes #10110 from brkyvz/batch-wal-test-fix.

(cherry picked from commit 6fd9e70)
Signed-off-by: Tathagata Das <[email protected]>
@asfgit asfgit closed this in 6fd9e70 Dec 7, 2015
@brkyvz brkyvz deleted the batch-wal-test-fix branch February 3, 2019 20:59
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.

4 participants