Todd Lipcon has submitted this change and it was merged. Change subject: service_queue: fixes based on Adar's post-commit review ......................................................................
service_queue: fixes based on Adar's post-commit review Adar left some notes on the LifoServiceQueue patch after it was committed here: http://gerrit.cloudera.org:8080/#/c/2938/7 This addresses those issues (mostly nits). Change-Id: Ib7f8d5205a319c1e0ddd1b01d63dac5d4430d5de Reviewed-on: http://gerrit.cloudera.org:8080/3026 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M build-support/tsan-suppressions.txt M src/kudu/rpc/service_queue-test.cc M src/kudu/rpc/service_queue.cc M src/kudu/rpc/service_queue.h 4 files changed, 31 insertions(+), 30 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3026 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib7f8d5205a319c1e0ddd1b01d63dac5d4430d5de Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
