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

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

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



##########
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:
       This will be used on HMS and occasionally on HS2 (if the Worker is HS2).
   Where are those other Compaction related configs? If everything is on HMS side, please move this there too




----------------------------------------------------------------
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