[ 
https://issues.apache.org/jira/browse/HIVE-19985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16560639#comment-16560639
 ] 

Gopal V commented on HIVE-19985:
--------------------------------

This patch is LGTM - +1 (after ORC release).

However, this impl does not apply to the setBaseAndInnerReader() passed over 
from the LLAP IO elevator case.

> ACID: Skip decoding the ROW__ID sections for read-only queries 
> ---------------------------------------------------------------
>
>                 Key: HIVE-19985
>                 URL: https://issues.apache.org/jira/browse/HIVE-19985
>             Project: Hive
>          Issue Type: Improvement
>          Components: Transactions
>            Reporter: Gopal V
>            Assignee: Eugene Koifman
>            Priority: Major
>              Labels: Branch3Candidate
>         Attachments: HIVE-19985.01.patch
>
>
> For a base_n file there are no aborted transactions within the file and if 
> there are no pending delete deltas, the entire ACID ROW__ID can be skipped 
> for all read-only queries (i.e SELECT), though it still needs to be projected 
> out for MERGE, UPDATE and DELETE queries.
> This patch tries to entirely ignore the ACID ROW__ID fields for all tables 
> where there are no possible deletes or aborted transactions for an ACID split.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to