Vectorized row batch object will represent the row batch that vectorized operators will work on. Refer to design spec attached to HIVE-4160 for details.
Implement classes for column vectors.
Committed to vectorization branch. Thanks, Eric!
Thanks Jitendra and Carl for reviewing!
New diff available for review at https://reviews.apache.org/r/10592/
modified patch with updates based on code review comments
Uploaded new diff to review tool at https://reviews.apache.org/r/10592/
Updated based on code review comments.
I left some comments on reviewboard.
Disregard above code review link. Instead please use this:
Code review link: https://reviews.facebook.net/differential/diff/32307/
removed unwanted .rej file
Hive's coding conventions are described here:
Please review this and correct the formatting issues in this patch.
Please limit the length of a line to 80 chars.
100 characters is acceptable.
new patch that addresses Jitendra's comments
Comments from Jitendra.
For a big patch like this, it would be very useful to have a phabricator entry
A few comments on the patch.
This is to be applied to the "vectorization" branch off the hive main trunk.
Adds VectorizedRowBatch class in org.apache.hadoop.hive.ql.exec.vector, plus unit test.