[GitHub] [arrow] WeichenXu123 commented on pull request #9187: ARROW-11223: [Java] Fix: BaseVariableWidthVector/BaseLargeVariableWidthVector setNull() and getBufferSizeFor() trigger offset buffer overflow

Sun, 31 Jan 2021 18:52:26 -0800 


WeichenXu123 commented on pull request #9187:
URL: https://github.com/apache/arrow/pull/9187#issuecomment-770528506


   I think we can simply the interface @BryanCutler proposed. We can make it to 
be static method, for example:
   
   For String/Binary type:
   
   `VarCharVector.getBufferSize(valueCount, avgValueBytesLen)`
   
   and provide a method to get average value bytes length:
   
   `varCharVecInstance.getAvgValueBytesLen()`
   
   For fixed width type, such as Int,
   
   `IntVector.getBufferSize(valueCount)`
   
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to