You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org> on 2016/12/01 12:19:04 UTC

[kudu-CR] Make linked list-test finish with snapshot scans in the present

David Ribeiro Alves has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/5306

Change subject: Make linked_list-test finish with snapshot scans in the present
......................................................................

Make linked_list-test finish with snapshot scans in the present

This makes linked_list-test finish WaitAndVerify with a snapshot
scan in the present. This was a long standing TODO that is now
possible due to safe time advancement. Note that finishing with
a READ_LATEST scan is still possible for the case where the a
majority is down and safe time doesn't advance.

Ran this on dist test with run config:
KUDU_ALLOW_SLOW_TESTS=1 ../../build-support/dist_test.py loop -n 500 bin/linked_list-test --stress_cpu_threads=4

Results:
linked_list-test http://dist-test.cloudera.org//job?job_id=david.alves.1480592295.28488

Change-Id: I50e8648c67a2f2970936e79548093241a6c36efc
---
M src/kudu/integration-tests/linked_list-test-util.h
M src/kudu/integration-tests/linked_list-test.cc
2 files changed, 79 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/5306/1
-- 
To view, visit http://gerrit.cloudera.org:8080/5306
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I50e8648c67a2f2970936e79548093241a6c36efc
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dr...@apache.org>

[kudu-CR] Make linked list-test finish with snapshot scans in the present

Posted by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org>.
David Ribeiro Alves has abandoned this change.

Change subject: Make linked_list-test finish with snapshot scans in the present
......................................................................


Abandoned

merged this with another patch

-- 
To view, visit http://gerrit.cloudera.org:8080/5306
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I50e8648c67a2f2970936e79548093241a6c36efc
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>