Hello Jean-Daniel Cryans, Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/7839
to review the following change.
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
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
1 file changed, 4 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/39/7839/1
--
To view, visit http://gerrit.cloudera.org:8080/7839
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I1791f5ed51e2d79cd9e9bcbf83e9b30eede871b4
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
