Hello Dan Burkert, Adar Dembo, I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4119 to review the following change. Change subject: java: fix leak of TabletClient objects in client2tablets map ...................................................................... java: fix leak of TabletClient objects in client2tablets map After running YCSB for a week, most of the clients had hit OOMEs. Some heap dump analysis showed that the client2tablets map had hundreds of thousands of leaked clients. It seems that we were neglecting to remove the client from the client2tablets map upon a disconnect. This fixes the issue and adds a regression test which reproduced the bug. Change-Id: I302650f2a6526e7c51537264137a4f00cbbda073 --- M java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java M java/kudu-client/src/test/java/org/apache/kudu/client/BaseKuduTest.java M java/kudu-client/src/test/java/org/apache/kudu/client/MiniKuduCluster.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java 4 files changed, 52 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/4119/1 -- To view, visit http://gerrit.cloudera.org:8080/4119 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I302650f2a6526e7c51537264137a4f00cbbda073 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com>