Hello Kudu Jenkins, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10199
to look at the new patch set (#3).
Change subject: java: fix minor synchronization issues exposed by error-prone
......................................................................
java: fix minor synchronization issues exposed by error-prone
* Unsynchronized access to 'sessions' in AsyncKuduClient.closeAllSessions().
This isn't likely to cause problems since it only happens on 'close'
and would only race if a new session was concurrently started.
* Add missing GuardedBy annotations in a few spots (non-functional)
* Fix synchronization in TableLocationsCache.toString() to properly
acquire the lock. Unlikely to cause problems in practice since we only
call toString() on this in some rare trace messages (if that).
* Fix synchronization in FakeDNS (test-only code)
Change-Id: I1c737f59928f393883d198872419e8832dfff006
---
M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
M java/kudu-client/src/main/java/org/apache/kudu/client/Connection.java
M java/kudu-client/src/main/java/org/apache/kudu/client/TableLocationsCache.java
M java/kudu-client/src/test/java/org/apache/kudu/client/FakeDNS.java
4 files changed, 16 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/10199/3
--
To view, visit http://gerrit.cloudera.org:8080/10199
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1c737f59928f393883d198872419e8832dfff006
Gerrit-Change-Number: 10199
Gerrit-PatchSet: 3
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
