[GitHub] dlg99 opened a new issue #1584: LedgerHandleAdv should expose asyncAddEntry variant that takes ByteBuf (LedgerHandle exposes it as public)

[GitBox <gi...@apache.org>]

dlg99 opened a new issue #1584: LedgerHandleAdv should expose asyncAddEntry variant that takes ByteBuf (LedgerHandle exposes it as public)
URL: https://github.com/apache/bookkeeper/issues/1584
 
 
   **FEATURE REQUEST**
   
   1. Please describe the feature you are requesting.
   
   LedgerHandle.java:
   ```java
       public void asyncAddEntry(ByteBuf data, final AddCallback cb, final Object ctx) {
   ...
   ```
   
   LedgerHandleAdv.java:
   
   ```java
       private void asyncAddEntry(final long entryId, ByteBuf data,
               final AddCallbackWithLatency cb, final Object ctx) {
   ...
   ```
   
   It's useful to have this exposed as public for clients to make use of netty's allocator
   
   2. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?
   
   must have
   
   3. Provide any additional detail on your proposed use case for this feature.
   
   I'll send a PR soon.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services