[ https://issues.apache.org/jira/browse/HIVE-9283?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14267877#comment-14267877 ]
Sergio Peña commented on HIVE-9283: ----------------------------------- Hi [~dongc] Is there a way we can use the following variable to get the metastore directory? ${hiveconf:hive.metastore.warehouse.dir} I see that QTestUtil.java defined that variable in: conf.setVar(HiveConf.ConfVars.METASTOREWAREHOUSE, (new Path(dfsUriString, "/build/ql/test/data/warehouse/")).toString()); > Improve encryption related test cases > ------------------------------------- > > Key: HIVE-9283 > URL: https://issues.apache.org/jira/browse/HIVE-9283 > Project: Hive > Issue Type: Sub-task > Reporter: Dong Chen > Assignee: Dong Chen > Fix For: encryption-branch > > Attachments: HIVE-9283.patch > > > NO PRECOMMIT TESTS > I found some test cases .q file could be improved by: > 1. change the table location from {{/user/hive/warehouse...}} to > {{/build/ql/test/data/warehouse/...}}. > The reason is that the default warehouse dir defined in QTestUtil is the > latter one, and the partial mask is based on it. I think it is better to make > test cases consistent with code. Also the .hive_staging location we want in > .out will be shown then. > 2. add cleanup at the end. > drop table and delete key. Otherwise, some cases will fail caused by cannot > create existed key. (Put in HIVE-9286) -- This message was sent by Atlassian JIRA (v6.3.4#6332)