----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26534/ -----------------------------------------------------------
(Updated 十月 10, 2014, 7:30 a.m.) Review request for hive. Changes ------- fix a null pointer issue if key is not setted. Repository: hive-git Description ------- In this patch, it will includes interfaces or abstract classes of generic Key, CryptoCodec, Encryptor and Decryptor. And the JCE AES implementation of the interfaces and abstract classes. Diffs (updated) ----- common/pom.xml ad9f6c0 common/src/java/org/apache/hadoop/hive/common/crypto/CipherSuite.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/CryptoCodec.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/CryptoCodecFactory.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/Decryptor.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/Encryptor.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/Key.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/aes/AesDecryptor.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/aes/AesEncryptor.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/aes/JceAesCbcCryptoCodec.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/aes/JceAesCryptoCodec.java PRE-CREATION common/src/java/org/apache/hadoop/hive/common/crypto/aes/JceAesCtrCryptoCodec.java PRE-CREATION common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 7f4afd9 common/src/test/org/apache/hadoop/hive/common/crypto/TestCipherSuite.java PRE-CREATION common/src/test/org/apache/hadoop/hive/common/crypto/TestKey.java PRE-CREATION common/src/test/org/apache/hadoop/hive/common/crypto/aes/TestAesCbcCryptoCodec.java PRE-CREATION common/src/test/org/apache/hadoop/hive/common/crypto/aes/TestAesCtrCryptoCodec.java PRE-CREATION common/src/test/org/apache/hadoop/hive/common/crypto/aes/TestAesEncryptor.java PRE-CREATION Diff: https://reviews.apache.org/r/26534/diff/ Testing ------- Testcases are included in this patch. Thanks, Xiaomeng Huang
