[ 
https://issues.apache.org/jira/browse/HIVE-9307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14270398#comment-14270398
 ] 

Dong Chen commented on HIVE-9307:
---------------------------------

Update patch V1. 

The variable is used in 3 statement: {{CREATE table sql}}, {{dfs command}}, and 
{{CRYPTO command}}. The first 2 already can handle this variable. For CRYPTO 
command, since we only use it for test, I added the handling in 
{{QTestUtil.executeTestCommand()}} for simplicity.

The output of 2 cases changed by 2 lines, which are the table location in 
create query in PREHOOK / POSTHOOK is masked. After double check, I think it is 
OK and I misunderstood before.

> Use metastore warehouse dir variable from conf instead of hard coded dir in 
> encryption test
> -------------------------------------------------------------------------------------------
>
>                 Key: HIVE-9307
>                 URL: https://issues.apache.org/jira/browse/HIVE-9307
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Dong Chen
>            Assignee: Dong Chen
>         Attachments: HIVE-9307.patch
>
>
> NO PRECOMMIT TESTS
> Use the following variable to get the metastore directory 
> $\{hiveconf:hive.metastore.warehouse.dir\} in test cases.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to