-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30593/
-----------------------------------------------------------

Review request for drill.


Bugs: DRILL-2151
    https://issues.apache.org/jira/browse/DRILL-2151


Repository: drill-git


Description
-------

emoved the line

    offsetVector.getMutator().setSafe( (thisIndex+1) * ${type.width}, 
outputStart + len);

from *VariableLengthVector.copyFromSafe()*

and refactored the call to *from.offsetVector.getAccessor()* to reuse the 
result.


Diffs
-----

  exec/java-exec/src/main/codegen/templates/VariableLengthVectors.java aa5b702 

Diff: https://reviews.apache.org/r/30593/diff/


Testing
-------

all unit tests pass
"Functional - Passing - new" passes on 171
the query described in DRILL-2151 runs fine now

I'm still waiting for 57 to test "TPCH SF100 Parquet"


Thanks,

abdelhakim deneche

Reply via email to