Will-Lo commented on code in PR #3931:
URL: https://github.com/apache/gobblin/pull/3931#discussion_r1573038969


##########
gobblin-cluster/src/main/java/org/apache/gobblin/cluster/GobblinClusterConfigurationKeys.java:
##########
@@ -50,6 +50,8 @@ public class GobblinClusterConfigurationKeys {
   public static final boolean DEFAULT_STANDALONE_CLUSTER_MODE = false;
   // Root working directory for Gobblin cluster
   public static final String CLUSTER_WORK_DIR = GOBBLIN_CLUSTER_PREFIX + 
"workDir";
+  // Root working dir without appending the application name, keeping 
CLUSTER_WORK_DIR property for backward compatibility
+  public static final String CLUSTER_ABSOLUTE_WORK_DIR = 
GOBBLIN_CLUSTER_PREFIX + "absolute.workDir";

Review Comment:
   the expectation here is that the exact workdir will have the job_name 
appended to it hence no chance of there being collisions, and easier to 
identify the job associated to it via GaaS



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to