Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5305
to look at the new patch set (#2).
Change subject: WIP KUDU-1127 Don't hang scanner threads waiting for safe time
......................................................................
WIP KUDU-1127 Don't hang scanner threads waiting for safe time
This makes it so we don't hang scanner threads on snapshot reads
when the difference between safe time and a requested timestamp
is bigger than the difference between 'now' and the client's
deadline. This makes the server return a Status::ServiceUnavailable()
with a THROTTLED error back to the client so that it can retry.
This allowed to swap linked_list-test to finish with snapshot scans
in the present and have it run on dist-test for 500 loops without
errors.
WIP as this requires a small directed test.
Change-Id: Ic7cd0b0749e715c5d9e665a8e37d0f1c95af574e
---
M src/kudu/consensus/time_manager.cc
M src/kudu/integration-tests/linked_list-test-util.h
M src/kudu/integration-tests/linked_list-test.cc
M src/kudu/tserver/tablet_service.cc
4 files changed, 158 insertions(+), 61 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/5305/2
--
To view, visit http://gerrit.cloudera.org:8080/5305
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic7cd0b0749e715c5d9e665a8e37d0f1c95af574e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
