> On Nov. 21, 2014, 8:25 p.m., Brock Noland wrote:
> > 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

Yes, the previous patch is not ready for review. And I have complete the patch 
and update this entry. This patch is tested locally by creating a simple 
database in the encrypted database. Please help me review it. Thanks!


> On Nov. 21, 2014, 8:25 p.m., Brock Noland wrote:
> > itests/qtest/pom.xml, line 546
> > <https://reviews.apache.org/r/28283/diff/2/?file=771833#file771833line546>
> >
> >     Encrypted is misspelled

change to encrption?


- cheng


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


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