[ARROW-5225] [Java] Improve performance of BaseValueVector#getValidityBufferSizeFromCount

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 0.14.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/21698
• Assignee: Ji Liu
• Reporter: Ji Liu
• Created: 28/Apr/19 03:22
• Updated: 11/Jan/23 07:39
• Resolved: 29/Apr/19 18:23

Description

Now in BaseValueVector#getValidityBufferSizeFromCount  and BitVectorHelper#getValidityBufferSize, it uses Math.ceil to calculate size which is not efficient (lots of unnecessary logic in StrictMath#floorOrCeil) . Since the valueCount is always not less than 0, we could simply replace Math.ceil with the following code:

return (valueCount+7) >> 3;

Issue Links

• links to: GitHub Pull Request #4218