You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/12/07 13:37:41 UTC
[GitHub] [bookkeeper] HQebupt opened a new pull request, #3695: [refactor][bookkeeper] Refactor ByteBuf release method in bookkeeper-common-allocator
HQebupt opened a new pull request, #3695:
URL: https://github.com/apache/bookkeeper/pull/3695
### Motivation
It may throw an exception when release a `ByteBuf` object. So the exception in `ByteBuf.release` should be checked.
### Changes
Use `ReferenceCountUtil.safeRelease()` instead of `ByteBuf.release()`.
Master Issue: [#7](https://github.com/HQebupt/bookkeeper/pull/7)
--
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: commits-unsubscribe@bookkeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [bookkeeper] StevenLuMT merged pull request #3695: [refactor][bookkeeper] Refactor ByteBuf release method in bookkeeper-common-allocator
Posted by GitBox <gi...@apache.org>.
StevenLuMT merged PR #3695:
URL: https://github.com/apache/bookkeeper/pull/3695
--
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: commits-unsubscribe@bookkeeper.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org