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