Sailesh Mukil has uploaded a new patch set (#3). Change subject: IMPALA-4880: Clarify synchronization policy for 'done_' in KuduScanNode ......................................................................
IMPALA-4880: Clarify synchronization policy for 'done_' in KuduScanNode This patch adds a comment to clarify that it's safe to optimistically read 'done_' in some places in KuduScanNode. This is to avoid some confusion as the code synchronizes access to the variable in certain places but not in others. Also, did some miscellaneous cleanup in a couple of places. Change-Id: I72c5f1fecf4bcf737d71a1fa13e59361604485f0 --- M be/src/exec/kudu-scan-node.cc M be/src/exec/kudu-scan-node.h 2 files changed, 11 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/94/5494/3 -- To view, visit http://gerrit.cloudera.org:8080/5494 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I72c5f1fecf4bcf737d71a1fa13e59361604485f0 Gerrit-PatchSet: 3 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Lars Volker <l...@cloudera.com> Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>