bbeaudreault commented on code in PR #4592:
URL: https://github.com/apache/hbase/pull/4592#discussion_r912359658
##########
hbase-common/src/main/java/org/apache/hadoop/hbase/nio/RefCnt.java:
##########
@@ -31,7 +34,10 @@
@InterfaceAudience.Private
public class RefCnt extends AbstractReferenceCounted {
- private Recycler recycler = ByteBuffAllocator.NONE;
+ private static final ResourceLeakDetector<RefCnt> detector =
Review Comment:
I wrote up that issue before Andrew asked to add here. I'm happy to just
close that issue if this lands here. Looks like this can land here easily, so
will just close that one once this PR is merged.
@virajjasani I think Andrew's comment here was asking if we should copy the
source code of ResourceLeakDetector into the hbase repo, to avoid having
upstream changes break the leak detection for us. But I could be wrong.
Personally I don't think it's worth copying it in -- Looks like it's been
around in netty for years and has been mostly stable, there was one minor
improvement to it recently. I'd rather only copy it in if we have a reason to,
like if we notice that they break it for us in later release. Maybe that's too
reactionary though.
I'm happy to go with consensus here, but if we decide to copy it in I think
we should maybe push this work to HBASE-27173 instead.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
