@reddycharan safeRelease() is not idempotent.
All it does is suppresses exception and logs it:

```
    public static void safeRelease(Object msg) {
        try {
            release(msg);
        } catch (Throwable t) {
            logger.warn("Failed to release a message: {}", msg, t);
        }
    }
```

The only "benefit" we are getting form it (vs .release()) is that it hides the 
problem.


[ Full content available at: https://github.com/apache/bookkeeper/pull/1735 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org

Reply via email to