[ https://issues.apache.org/jira/browse/HIVE-9995?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16798372#comment-16798372 ]
Hive QA commented on HIVE-9995: ------------------------------- Here are the results of testing the latest attachment: https://issues.apache.org/jira/secure/attachment/12963282/HIVE-9995.04.patch {color:green}SUCCESS:{color} +1 due to 2 test(s) being added or modified. {color:green}SUCCESS:{color} +1 due to 15833 tests passed Test results: https://builds.apache.org/job/PreCommit-HIVE-Build/16615/testReport Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/16615/console Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-16615/ Messages: {noformat} Executing org.apache.hive.ptest.execution.TestCheckPhase Executing org.apache.hive.ptest.execution.PrepPhase Executing org.apache.hive.ptest.execution.YetusPhase Executing org.apache.hive.ptest.execution.ExecutionPhase Executing org.apache.hive.ptest.execution.ReportingPhase {noformat} This message is automatically generated. ATTACHMENT ID: 12963282 - PreCommit-HIVE-Build > ACID compaction tries to compact a single file > ---------------------------------------------- > > Key: HIVE-9995 > URL: https://issues.apache.org/jira/browse/HIVE-9995 > Project: Hive > Issue Type: Bug > Components: Transactions > Affects Versions: 1.0.0 > Reporter: Eugene Koifman > Assignee: Denys Kuzmenko > Priority: Major > Attachments: HIVE-9995.01.patch, HIVE-9995.02.patch, > HIVE-9995.03.patch, HIVE-9995.04.patch, HIVE-9995.WIP.patch > > > Consider TestWorker.minorWithOpenInMiddle() > since there is an open txnId=23, this doesn't have any meaningful minor > compaction work to do. The system still tries to compact a single delta file > for 21-22 id range, and effectively copies the file onto itself. > This is 1. inefficient and 2. can potentially affect a reader. > (from a real cluster) > Suppose we start with > {noformat} > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:03 > /user/hive/warehouse/t/base_0000016 > -rw-r--r-- 1 ekoifman staff 602 2016-06-09 16:03 > /user/hive/warehouse/t/base_0000016/bucket_00000 > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:07 > /user/hive/warehouse/t/base_0000017 > -rw-r--r-- 1 ekoifman staff 588 2016-06-09 16:07 > /user/hive/warehouse/t/base_0000017/bucket_00000 > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:07 > /user/hive/warehouse/t/delta_0000017_0000017_0000 > -rw-r--r-- 1 ekoifman staff 514 2016-06-09 16:06 > /user/hive/warehouse/t/delta_0000017_0000017_0000/bucket_00000 > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:07 > /user/hive/warehouse/t/delta_0000018_0000018_0000 > -rw-r--r-- 1 ekoifman staff 612 2016-06-09 16:07 > /user/hive/warehouse/t/delta_0000018_0000018_0000/bucket_00000 > {noformat} > then do _alter table T compact 'minor';_ > then we end up with > {noformat} > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:07 > /user/hive/warehouse/t/base_0000017 > -rw-r--r-- 1 ekoifman staff 588 2016-06-09 16:07 > /user/hive/warehouse/t/base_0000017/bucket_00000 > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:11 > /user/hive/warehouse/t/delta_0000018_0000018 > -rw-r--r-- 1 ekoifman staff 500 2016-06-09 16:11 > /user/hive/warehouse/t/delta_0000018_0000018/bucket_00000 > drwxr-xr-x - ekoifman staff 0 2016-06-09 16:07 > /user/hive/warehouse/t/delta_0000018_0000018_0000 > -rw-r--r-- 1 ekoifman staff 612 2016-06-09 16:07 > /user/hive/warehouse/t/delta_0000018_0000018_0000/bucket_00000 > {noformat} > So compaction created a new dir _/user/hive/warehouse/t/delta_0000018_0000018_ -- This message was sent by Atlassian JIRA (v7.6.3#76005)