horizonzy opened a new issue, #3428:
URL: https://github.com/apache/bookkeeper/issues/3428

   **BUG REPORT**
   When we get entry from bookie in ReadEntryProcessorV3#readEntry, the 
response is DirectByteBuf, the v3 use proto buf response. So it will copy byte 
array to proto buf builder, it will use heap memory. 
   
   If the user read request is frequent, it will introduce gc pressure.
   
   
https://github.com/apache/bookkeeper/blob/ca7d7c23e3aa572befeb447157b3ef9d96bb1449/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ReadEntryProcessorV3.java#L179
   
   ```
       public static ByteString copyFrom(ByteBuffer bytes, int size) {
           checkRange(0, size, bytes.remaining());
           byte[] copy = new byte[size];
           bytes.get(copy);
           return new LiteralByteString(copy);
       }
   ```
   
   


-- 
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: issues-unsubscr...@bookkeeper.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to