[ARROW-6200] [Java] Method getBufferSizeFor in BaseRepeatedValueVector/ListVector not correct

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22591
• Assignee: Ji Liu
• Reporter: Ji Liu
• Created: 11/Aug/19 12:13
• Updated: 11/Jan/23 07:45
• Resolved: 12/Aug/19 08:54

Description

Currently, getBufferSizeFor in BaseRepeatedValueVector implemented as below:

if (valueCount == 0) {

  return 0;

}

return ((valueCount + 1) * OFFSET_WIDTH) + vector.getBufferSizeFor(valueCount);

Here vector.getBufferSizeFor(valueCount) seems not right which should be

 

int innerVectorValueCount = offsetBuffer.getInt(valueCount * OFFSET_WIDTH);

vector.getBufferSizeFor(innerVectorValueCount)

 ListVector has the same problem.

Issue Links

• links to: GitHub Pull Request #5060