[GitHub] [hive] klcopp commented on a change in pull request #1518: HIVE-24191: Introduce configurable user to run compaction as

[GitBox <gi...@apache.org>]

klcopp commented on a change in pull request #1518:
URL: https://github.com/apache/hive/pull/1518#discussion_r493520605



##########
File path: common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
##########
@@ -3098,6 +3098,10 @@ private static void populateLlapDaemonVarsSet(Set<String> llapDaemonVarsSetLocal
         "This is set to compactor from within the query based compactor. This enables the Tez SplitGrouper "
         + "to group splits based on their bucket number, so that all rows from different bucket files "
         + " for the same bucket number can end up in the same bucket file after the compaction."),
+    // MetastoreConf doesn't have a restricted list. Otherwise can be moved there.
+    COMPACTOR_RUN_AS_USER("hive.compactor.run.as.user", "",
+        "Specify the user to run compaction (Worker) and compaction-related file listing (Initiator, Cleaner) as. " +
+        "If empty string, defaults to table/partition directory owner. Restricted."),

Review comment:
       MetastoreConf, but it has no restricted list AFAIK. Implementing one would be a new feature that isn't covered by this ticket.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org