> On Dic. 1, 2014, 9:35 p.m., Sergio Pena wrote: > > itests/util/src/main/java/org/apache/hadoop/hive/ql/QTestUtil.java, line 268 > > <https://reviews.apache.org/r/28283/diff/3/?file=778006#file778006line268> > > > > Do we need to set this value? For what I know, AES/CTR/NoPadding is the > > only cipher mode that HDFS supports. > > cheng xu wrote: > Yes, you are right. We can remove it at this point. I add the setter here > just in case one or more cipher will be supported later.
Thanks. I don't think hdfs will add new ciphers in the near future. CTR is the only cipher mode that fits with how files are stored on hdfs, and it is very secure, and AES is the best approved algorithm by PCI & HIPAA these days. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28283/#review63437 ----------------------------------------------------------- On Dic. 2, 2014, 2:58 a.m., cheng xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28283/ > ----------------------------------------------------------- > > (Updated Dic. 2, 2014, 2:58 a.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > The patch includes: > 1. enable security properties for hive security cluster > > > Diffs > ----- > > .gitignore c5decaf > data/scripts/q_test_cleanup_for_encryption.sql PRE-CREATION > data/scripts/q_test_init_for_encryption.sql 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 > ql/src/test/queries/clientpositive/create_encrypted_table.q PRE-CREATION > 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 > shims/common/src/main/java/org/apache/hadoop/hive/shims/HadoopShims.java > fa66a4a > > Diff: https://reviews.apache.org/r/28283/diff/ > > > Testing > ------- > > > Thanks, > > cheng xu > >
