hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Barna Zsombor Klara <zsombor.kl...@cloudera.com>
Subject Review Request 61656: HIVE-17316: Use regular expressions for the hidden configuration variables
Date Tue, 15 Aug 2017 13:33:39 GMT

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

Review request for hive and Peter Vary.


Repository: hive-git


Description
-------

HIVE-17316: Use regular expressions for the hidden configuration variables


Diffs
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 056f2d78346b6b306d34dfb610e3a7fed4ca68aa

  common/src/java/org/apache/hadoop/hive/conf/HiveConfUtil.java 9084fed9fa5121285709874923b3e03e887cb20d

  common/src/test/org/apache/hadoop/hive/conf/TestHiveConfRestrictList.java 980fa761541d72a0b37bc5d687e1fbc4551b5282

  common/src/test/org/apache/hadoop/hive/conf/TestHiveConfUntil.java PRE-CREATION 


Diff: https://reviews.apache.org/r/61656/diff/1/


Testing
-------

Added unit tests and checked that the current set of restricted variables will not restrict
any new ones due to the change in the check from equals to contains.


Thanks,

Barna Zsombor Klara


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message