hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sergio Peña (JIRA) <j...@apache.org>
Subject [jira] [Commented] (HIVE-9283) Improve encryption related test cases
Date Wed, 07 Jan 2015 17:09:35 GMT

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

Mime
View raw message