Aggarwal-Raghav opened a new pull request, #5909:
URL: https://github.com/apache/hive/pull/5909

   ### What changes were proposed in this pull request?
   [HIVE-29053](https://issues.apache.org/jira/browse/HIVE-29053), Skip the 
LOCATION field for proto tables when running qtests.
   
   
   ### Why are the changes needed?
   Running the qtest containing `--! qt:sysdb`, causes creation of following 
dir in itests/qtest directory.
   ```
   _REPLACE_WITH_APP_DATA_LOCATION_
   _REPLACE_WITH_DAG_DATA_LOCATION_
   _REPLACE_WITH_DAG_META_LOCATION_
   _REPLACE_WITH_QUERY_DATA_LOCATION_
   ```
   
   It better to have the proto tables share same location as other sys tables 
for testing/debugging purposes. 
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   ### How was this patch tested?
   Ran the following command with and without the PR and observed the directory 
creation in `itests/qtest` directory
   ```
   mvn clean test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=query_history.q 
-Drat.skip -pl itests/qtest -Pitests
   mvn clean test -Dtest=TestMiniLlapLocalCliDriver -Dqfile=sysdb.q -Drat.skip 
-pl itests/qtest -Pitests
   ```


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