Sailesh Mukil has posted comments on this change. Change subject: KUDU-1865 (part 1): reduce some cross-thread allocations ......................................................................
Patch Set 1: > (2 comments) > > I found a couple of bugs, but there are still some memory leaks I > haven't been able to figure out. This now passes DEBUG testing. > I'll keep at it. I looked at this a bit more and it seems that the leaks are coming from ThreadRestrictions::LoadTLS() and KernelStackWatchdog::GetTLS(). Any idea how there could be leaks there, since that code hasn't changed for this patch? -- To view, visit http://gerrit.cloudera.org:8080/5905 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7d4d5f14fb302196b1797c712b21cfce81f157c1 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Tidy Bot Gerrit-HasComments: No