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

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

Patch attached for review. Still not good enough.

When using $\{hiveconf:hive.metastore.warehouse.dir\} instead of 
/build/ql/test/data/warehouse, there is a slight difference. The variable value 
is  {{*hdfs://localhost:37553*/build/ql/test/data/warehouse}}. A schema and 
network address is added.
The test still works, but diff failed. The result log changes a little by mask 
pattern. But I think the masked line is useful and should not be masked. Still 
thinking a better solution to fix this.

In order not to impact our branch test today, let's not merge it for now.

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