Adar Dembo has posted comments on this change. Change subject: [java client] Cleanup AsyncKuduClient's unused caches ......................................................................
Patch Set 4: > > So no regression test for that race? > > Sorry, where in this gerrit did you mention a regression test? :P I didn't mention it in the gerrit, but it's what we discussed on Slack. I assumed you'd pair it with this patch since without the patch, this new test would fail. > TBH right now it'd be a pain. Right now I'm doing even more > refactoring, I think testing will soon become a lot easier. Okay. -- To view, visit http://gerrit.cloudera.org:8080/4705 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I62802c34c618c83a4ff69d79825387cbe4ab51a8 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans <[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 Gerrit-HasComments: No
