Dan Burkert has submitted this change and it was merged. Change subject: KUDU-1544: Race in Java client's AsyncKuduSession.apply() ......................................................................
KUDU-1544: Race in Java client's AsyncKuduSession.apply() This fixes a potential race in AsyncKuduSession.apply by acquiring the notification deferred early. See the JIRA for details about the race. Acquiring the notification deferred early is always safe, the only downside is that clients may be spuriously notified that there is buffer space available when in fact there is not. Change-Id: I1791f5ed51e2d79cd9e9bcbf83e9b30eede871b4 Reviewed-on: http://gerrit.cloudera.org:8080/7839 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/7839 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1791f5ed51e2d79cd9e9bcbf83e9b30eede871b4 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins
