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

(Updated Aug. 31, 2016, 3:32 p.m.)


Review request for hive, Chaoyu Tang, Gabor Szadovszky, Sergio Pena, and Barna 
Zsombor Klara.


Bugs: HIVE-14426
    https://issues.apache.org/jira/browse/HIVE-14426


Repository: hive-git


Description
-------

Used HiveConf.stripHiddenConfigurations to remove sensitive information from 
configuration dump. Had to refactor, so it could be applied to simple 
Configuration objects too, like AppConfig.

Used Utilities.maskIfPassword to remove sensitive information from property dump


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/LogUtils.java 599e798 
  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 14a538b 
  common/src/java/org/apache/hadoop/hive/conf/HiveConfUtil.java 073a978 
  common/src/test/org/apache/hadoop/hive/common/TestLogUtils.java PRE-CREATION 
  
hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/AppConfig.java
 dd1208b 
  
hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/tool/TempletonUtils.java
 201e647 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java a542dc4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java ac922ce 
  ql/src/java/org/apache/hadoop/hive/ql/exec/spark/HiveSparkClientFactory.java 
8e5c0da 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestUtilities.java 0786f9b 

Diff: https://reviews.apache.org/r/51435/diff/


Testing
-------

Manually - checked both


Thanks,

Peter Vary

Reply via email to