[jira] [Comment Edited] (HIVE-20635) VectorizedOrcAcidRowBatchReader doesn't filter delete events for original files

["Eugene Koifman (JIRA)" <ji...@apache.org>]

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

Eugene Koifman edited comment on HIVE-20635 at 10/4/18 11:11 PM:
-----------------------------------------------------------------

A nit: {{LOG.info("findOriginalMinMaxKeys(): " + keyIntervalTmp);}} is only logged {{if (minRowId >= maxRowId) {}}

would be better to log it always

otherwise LGTM


was (Author: ekoifman):
A nit: {{      LOG.info("findOriginalMinMaxKeys(): " + keyIntervalTmp);}} is only logged {{if (minRowId >= maxRowId) {}}

would be better to log it always

otherwise LGTM

> VectorizedOrcAcidRowBatchReader doesn't filter delete events for original files
> -------------------------------------------------------------------------------
>
>                 Key: HIVE-20635
>                 URL: https://issues.apache.org/jira/browse/HIVE-20635
>             Project: Hive
>          Issue Type: Improvement
>          Components: Transactions
>    Affects Versions: 3.0.0
>            Reporter: Eugene Koifman
>            Assignee: Saurabh Seth
>            Priority: Major
>         Attachments: HIVE-20635.2.patch, HIVE-20635.patch
>
>
> this is a followup to HIVE-16812 which adds support for delete event filtering for splits from native acid files
> need to add the same for {{OrcSplit.isOriginal()}}



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