[ https://issues.apache.org/jira/browse/HIVE-20823?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16765829#comment-16765829 ]
Abhishek Somani commented on HIVE-20823: ---------------------------------------- [~ekoifman], [~vgumashta] Is there a plan to backport this to the 3.x branch? > Make Compactor run in a transaction > ----------------------------------- > > Key: HIVE-20823 > URL: https://issues.apache.org/jira/browse/HIVE-20823 > Project: Hive > Issue Type: Sub-task > Components: Transactions > Affects Versions: 3.0.0 > Reporter: Eugene Koifman > Assignee: Eugene Koifman > Priority: Major > Fix For: 4.0.0 > > Attachments: HIVE-20823.01.patch, HIVE-20823.03.patch, > HIVE-20823.04.patch, HIVE-20823.05.patch, HIVE-20823.07.patch, > HIVE-20823.08.patch, HIVE-20823.09.patch, HIVE-20823.10.patch, > HIVE-20823.11.patch, HIVE-20823.11.patch, HIVE-20823.12.patch, > HIVE-20823.13.patch, HIVE-20823.14.patch > > > Have compactor open a transaction and run the job in that transaction. > # make compactor produced base/delta include this txn id in the folder name, > e.g. base_7_c17 where 17 is the txnid. > # add {{CQ_TXN_ID bigint}} to COMPACTION_QUEUE and COMPLETED_COMPACTIONS to > record this txn id > # make sure {{AcidUtils.getAcidState()}} pays attention to this transaction > on read and ignores this dir if this txn id is not committed in the current > snapshot > ## this means not only validWriteIdList but ValidTxnIdList should be passed > along in config (if it isn't yet) > # once this is done, {{CompactorMR.createCompactorMarker()}} can be > eliminated and {{AcidUtils.isValidBase}} modified accordingly > # modify Cleaner so that it doesn't clean old files until new file is visible > to all readers > # -- This message was sent by Atlassian JIRA (v7.6.3#76005)