Hello Adar Dembo,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: util: add a deadline to BlockingQueue::BlockingPut()
......................................................................

util: add a deadline to BlockingQueue::BlockingPut()

Drawing inspiration from BlockingDrainTo(), this adds a deadline to
BlockingPut(), with the same error semantics.

This also updates SubprocessServer to use the the new method, instead of
manual deadline-checking.

Change-Id: If32bad5667195d72eeb03e4942751824d4dadb7a
---
M src/kudu/consensus/log.cc
M src/kudu/subprocess/server.cc
M src/kudu/subprocess/subprocess_server-test.cc
M src/kudu/util/blocking_queue-test.cc
M src/kudu/util/blocking_queue.h
5 files changed, 50 insertions(+), 39 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: If32bad5667195d72eeb03e4942751824d4dadb7a
Gerrit-Change-Number: 15324
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>

Reply via email to