Public
  1. Public

arrow

Public

Commits

tianchenPindikura Ravindra
6c1fccdb5bb
tianchen authored and Pindikura Ravindra committed 6381043a6ea
ARROW-6200: [Java] Method getBufferSizeFor in BaseRepeatedValueVector/ListVector not correct

Related to ARROW-6200.

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

Closes #5060 from tianchen92/ARROW-6200 and squashes the following commits:

f0bdbfb69 <tianchen> reuse code
90900e785 <tianchen> ARROW-6200:  Method getBufferSizeFor in BaseRepeatedValueVector/ListVector not correct

Authored-by: tianchen <niki.lj@alibaba-inc.com>
Signed-off-by: Pindikura Ravindra <ravindra@dremio.com>