hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brock Noland (JIRA)" <>
Subject [jira] [Commented] (HIVE-9283) Improve encryption related test cases
Date Wed, 07 Jan 2015 22:56:36 GMT


Brock Noland commented on HIVE-9283:

Using that variable is a good idea. I will commit this so we can get a clean run and [~dongc]
can you update the tests in a follow-on patch?

> Improve encryption related test cases
> -------------------------------------
>                 Key: HIVE-9283
>                 URL:
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Dong Chen
>            Assignee: Dong Chen
>             Fix For: encryption-branch
>         Attachments: HIVE-9283.patch
> 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

View raw message