Aggarwal-Raghav commented on code in PR #6228:
URL: https://github.com/apache/hive/pull/6228#discussion_r2607096318


##########
ql/src/test/results/clientpositive/llap/catalog.q.out:
##########
@@ -16,8 +16,6 @@ PREHOOK: Input: catalog:test_cat
 POSTHOOK: query: DESC CATALOG test_cat
 POSTHOOK: type: DESCCATALOG
 POSTHOOK: Input: catalog:test_cat
-Catalog Name   test_cat         

Review Comment:
   **Context:** After the java changes, the output of `desc catalog` i.e whole 
line is masked. Because of presence of `/tmp/`. With hdfs it is just masking 
the location instead of whole line
   
   
https://github.com/apache/hive/blob/62402e9edae3a5d5335545f6841725642120b88a/itests/util/src/main/java/org/apache/hadoop/hive/ql/QOutProcessor.java#L118
   
   `dfs -mkdir -p` is not working for TestMiniLlapLocalCliDriver. The q files 
(external_table_purge.q, ) in which it is working was in LlapCliDriver hence I 
moved the test.



-- 
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