[I] [Java] ArrowRecordBatch Body Length Computation Only Accounts for Last Buffer [arrow-java]

[via GitHub]

ParthChonkar opened a new issue, #188:
URL: https://github.com/apache/arrow-java/issues/188

   ### Describe the bug, including details regarding any error messages, 
version, and platform.
   
   Hi Arrow Community,
   
   I believe `ArrowRecordBatch::computeBodyLength` has an implementation bug. 
It does the following 
   
   ```
       for (int i = 0; i < buffers.size(); i++) {
         ArrowBuf buffer = buffers.get(i);
         ArrowBuffer layout = buffersLayout.get(i);
         size = layout.getOffset() + buffer.readableBytes();
   
         // round up size to the next multiple of 8
         size = DataSizeRoundingUtil.roundUpTo8Multiple(size);
       }
       return size;
   ```
   
   Since `size` is being assigned and not incremented at first - does this mean 
computeBodyLength only accounts for the last buffer ?
   
   ### Component(s)
   
   Java


-- 
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...@arrow.apache.org.apache.org

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