shivjha30 commented on code in PR #5000:
URL: https://github.com/apache/hive/pull/5000#discussion_r1547115174


##########
ql/src/test/org/apache/hadoop/hive/ql/TestTxnLoadData.java:
##########
@@ -394,16 +394,16 @@ public void loadDataPartitioned() throws Exception {
    */
   @Test
   public void testValidations() throws Exception {
-    runStatementOnDriver("drop table if exists T");
-    runStatementOnDriver("drop table if exists Tstage");

Review Comment:
   when we create a table **Tstage**, in warehouse directory the table will be 
created in small case letters like "tstage". While we are performing Load Data 
we are appending warehousedir + "Tstage"  in 
https://github.com/apache/hive/blob/0bc624ade3020826b4e7101b7ea4789be7e8585b/ql/src/test/org/apache/hadoop/hive/ql/TestTxnLoadData.java#L406
   
   As Linux is case sensitive, it couldnt find the path of warehousedir + 
"Tstage" but there is a path of warehousedir + "tstage" exists.
   
   Previously, the file path wasn't being validated in the Hive server. Now, 
after implementing a fix, the test has failed because the Hive server is now 
validating whether the file path exists or not.



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to