pvary commented on a change in pull request #2799:
URL: https://github.com/apache/hive/pull/2799#discussion_r759107544



##########
File path: 
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/txn/compactor/TestCompactor.java
##########
@@ -2314,6 +2315,10 @@ public void testCompactionOnDataLoadedInPath() throws 
Exception {
 
     executeStatementOnDriver("load data inpath '" + path002.toString() + "' 
into table comp3", driver);
     executeStatementOnDriver("load data inpath '" + 
path002.getParent().toString() + "' into table comp3", driver);
+    Table table3 = hmsClient.getTable("default", "comp3");
+    Path delta2bucket0  = fs.globStatus(new Path(table3.getSd().getLocation(), 
"delta_0000002_0000002_0000/000000_0"))[0].getPath();
+    Path emptyBucket = new Path(delta2bucket0.toString().replace("000000_0", 
"000003_0"));
+    fs.create(emptyBucket, true);

Review comment:
       Who would create an empty file in an ACID table?
   It is a very important precondition that nobody should mess around the files 
under an ACID table directory.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to