----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61656/ -----------------------------------------------------------
(Updated Aug. 15, 2017, 3:30 p.m.) Review request for hive and Peter Vary. Changes ------- Fixed failing tests and updated based on review comments. Thank you Peter for the review! Repository: hive-git Description ------- HIVE-17316: Use regular expressions for the hidden configuration variables Diffs (updated) ----- 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/TestHiveConfUtil.java PRE-CREATION service/src/test/org/apache/hive/service/cli/session/TestSessionGlobalInitFile.java 5760e66786b1521e439f7abc3597776449b6336d Diff: https://reviews.apache.org/r/61656/diff/2/ Changes: https://reviews.apache.org/r/61656/diff/1-2/ 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
