lhotari commented on a change in pull request #10480:
URL: https://github.com/apache/pulsar/pull/10480#discussion_r626396797



##########
File path: 
managed-ledger/src/main/java/org/apache/bookkeeper/mledger/util/RangeCache.java
##########
@@ -90,7 +90,7 @@ public Value get(Key key) {
             try {
                 value.retain();
                 return value;
-            } catch (Throwable t) {
+            } catch (IllegalReferenceCountException e) {

Review comment:
       yes, logging would be useful to get more information. I'm just wondering 
if it should be done only at debug level since it's not a real problem. It's 
part of the expected behavior that this could sometimes happen.




-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to