Repository: hive
Updated Branches:
  refs/heads/master 4df092674 -> 4bba139d3


HIVE-17690 : Add distcp.options.p* in sql standard authorization config 
whitelist (Thejas Nair, reviewed by Vaibhav Gumashta)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/4bba139d
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/4bba139d
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/4bba139d

Branch: refs/heads/master
Commit: 4bba139d3719c11a015919c1560ac473651f93c5
Parents: 4df0926
Author: Thejas M Nair <[email protected]>
Authored: Wed Oct 4 17:02:14 2017 -0700
Committer: Thejas M Nair <[email protected]>
Committed: Wed Oct 4 17:02:14 2017 -0700

----------------------------------------------------------------------
 common/src/java/org/apache/hadoop/hive/conf/HiveConf.java         | 2 +-
 .../plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java          | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/4bba139d/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
----------------------------------------------------------------------
diff --git a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 
b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
index ce391fd..da30b37 100644
--- a/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
+++ b/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
@@ -4512,7 +4512,7 @@ public class HiveConf extends Configuration {
     "distcp\\.options\\.overwrite",
     "distcp\\.options\\.strategy",
     "distcp\\.options\\.i",
-    "distcp\\.options\\.p",
+    "distcp\\.options\\.p.*",
     "distcp\\.options\\.update",
     "distcp\\.options\\.delete",
     "mapred\\.map\\..*",

http://git-wip-us.apache.org/repos/asf/hive/blob/4bba139d/ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
----------------------------------------------------------------------
diff --git 
a/ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
 
b/ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
index 531f4bb..c01dbeb 100644
--- 
a/ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
+++ 
b/ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
@@ -90,7 +90,8 @@ public class TestSQLStdHiveAccessControllerHS2 {
         "hive.optimize.index.filter.compact.maxsize", "hive.tez.dummy", 
"tez.task.dummy",
         "hive.exec.dynamic.partition", "hive.exec.dynamic.partition.mode",
         "hive.exec.max.dynamic.partitions", 
"hive.exec.max.dynamic.partitions.pernode",
-        "oozie.HadoopAccessorService.created", 
"tez.queue.name","hive.druid.select.distribute");
+        "oozie.HadoopAccessorService.created", 
"tez.queue.name","hive.druid.select.distribute",
+        "distcp.options.px");
 
     // combine two lists
     List<String> varList = new ArrayList<String>();

Reply via email to