-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28283/#review62626
-----------------------------------------------------------


Hi Ferdindand,

I think at the end of this JIRA we only want one q-file tests enabled. The 
other sub-tasks of HIVE-8065 are for enabling tests. I also think that as part 
of this test we'll need to startup a MiniDFS instance.

Brock


data/conf/encrypted/hive-site.xml
<https://reviews.apache.org/r/28283/#comment104702>

    Do we actually need a new hive-site?



itests/qtest/pom.xml
<https://reviews.apache.org/r/28283/#comment104700>

    Encrypted is misspelled



itests/src/test/resources/testconfiguration.properties
<https://reviews.apache.org/r/28283/#comment104701>

    Actually I think we want to support just a few q-file tests. Specifically 
we should only be enabling a few q-file tests for each sub-task here:
    
    https://issues.apache.org/jira/browse/HIVE-8065


- Brock Noland


On Nov. 21, 2014, 2:30 a.m., cheng xu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28283/
> -----------------------------------------------------------
> 
> (Updated Nov. 21, 2014, 2:30 a.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> The patch includes:
> 1. enable security properties for hive security cluster
> 
> 
> Diffs
> -----
> 
>   data/conf/encrypted/hive-site.xml PRE-CREATION 
>   itests/qtest/pom.xml 376f4a9 
>   itests/src/test/resources/testconfiguration.properties 3ae001d 
>   itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java 31d5c29 
>   shims/0.20S/src/main/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java 
> 2e00d93 
>   shims/0.23/src/main/java/org/apache/hadoop/hive/shims/Hadoop23Shims.java 
> 8161fc1 
> 
> Diff: https://reviews.apache.org/r/28283/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> cheng xu
> 
>

Reply via email to