Details
-
Type:
Sub-task
-
Status: Resolved
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: vectorization-branch
-
Fix Version/s: vectorization-branch, 0.13.0
-
Component/s: None
-
Labels:None
Description
Change ORC Record reader and Tree readers to return a set of Rows instead of a row.
Activity
Review available at https://reviews.apache.org/r/10712/
Show
Sarvesh Sakalanaga
added a comment - Review available at https://reviews.apache.org/r/10712/
Addressed comments from Scott and Eric. Review at https://reviews.apache.org/r/10712/diff/#index_header
Show
Sarvesh Sakalanaga
added a comment - Addressed comments from Scott and Eric. Review at https://reviews.apache.org/r/10712/diff/#index_header
Patch is not applying cleanly. Can you please rebase it?
Show
Ashutosh Chauhan
added a comment - Patch is not applying cleanly. Can you please rebase it?
Sure. Attached a new patch which is rebased.
Show
Sarvesh Sakalanaga
added a comment - Sure. Attached a new patch which is rebased.
Hi Ashutosh,
Sorry about that. Can you try the one attached?
Thanks,
Sarvesh
Show
Sarvesh Sakalanaga
added a comment - Hi Ashutosh,
Sorry about that. Can you try the one attached?
Thanks,
Sarvesh
Hi Ashutosh,
Sorry about that. Can you try the one attached?
Thanks,
Sarvesh
Show
Sarvesh Sakalanaga
added a comment - Hi Ashutosh,
Sorry about that. Can you try the one attached?
Thanks,
Sarvesh
Patch checked-in vector branch. Thanks, Sarvesh!
Show
Ashutosh Chauhan
added a comment - Patch checked-in vector branch. Thanks, Sarvesh!
The patch contains changes to ORC reader to return a batch of rows instead of a row. A new method called nextBatch() is added to ORC reader and tree readers of ORC. Currently only int,long,short,double,float,string and struct datatypes support batch processing.