[
https://issues.apache.org/jira/browse/DRILL-5602?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Paul Rogers updated DRILL-5602:
-------------------------------
Comment: was deleted
(was: The problem does not occur in the other forms of {{allocateNew}}, which
is why the problem has not often been seen. From {{VarCharVector}}:
{code}
@Override
public boolean allocateNewSafe() {
...
data = allocator.buffer(requestedSize);
allocationSizeInBytes = requestedSize;
offsetVector.allocateNew();
...
data.readerIndex(0);
offsetVector.zeroVector(); // <-- Zeros whole vector
return true;
}
{code}
)
> Repeated List Vector fails to initialize the offset vector
> ----------------------------------------------------------
>
> Key: DRILL-5602
> URL: https://issues.apache.org/jira/browse/DRILL-5602
> Project: Apache Drill
> Issue Type: Bug
> Affects Versions: 1.10.0
> Reporter: Paul Rogers
> Assignee: Paul Rogers
> Fix For: 1.11.0
>
>
> The code that allocates a new {{RepeatedListVector}} does not initialize the
> first offset to zero as required:
> {code}
> @Override
> public void allocateNew(int valueCount, int innerValueCount) {
> clear();
> getOffsetVector().allocateNew(valueCount + 1);
> getMutator().reset();
> }
> {code}
> Since Netty does not zero-fill vectors, the result is vector corruption.
> If the code worked correctly, here is the behavior when writing to the first
> element of the list:
> * Access the offset vector at offset 0. Should be 0.
> * Write the new value at that offset. Since the first offset is 0, the first
> value is written at 0 in the value vector.
> * Write into offset 1 the value at offset 0 plus the length of the new value.
> But, the offset vector is not initialized to zero. Instead, offset 0 contains
> the value 16 million. Now:
> * Access the offset vector at offset 0. Value is 16 million.
> * Write the new value at that offset. Write at position 16 million. This
> requires growing the value vector from its present size to 16 MB.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
