David Ribeiro Alves has submitted this change and it was merged. 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. This patch was worked on by Todd Lipcon and David Alves. Change-Id: I302650f2a6526e7c51537264137a4f00cbbda073 Reviewed-on: http://gerrit.cloudera.org:8080/4119 Tested-by: David Ribeiro Alves <dral...@apache.org> Reviewed-by: Todd Lipcon <t...@apache.org> --- 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, 72 insertions(+), 5 deletions(-) Approvals: David Ribeiro Alves: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/4119 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I302650f2a6526e7c51537264137a4f00cbbda073 Gerrit-PatchSet: 4 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> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>