kw2542 commented on a change in pull request #1238: SAMZA-2407: Add 
job.config.loader.* configs in JobConfig & corresponding env variables in 
ShellCommandConfig
URL: https://github.com/apache/samza/pull/1238#discussion_r364973554
 
 

 ##########
 File path: 
samza-core/src/main/scala/org/apache/samza/config/ShellCommandConfig.scala
 ##########
 @@ -25,6 +25,11 @@ object ShellCommandConfig {
    */
   val ENV_COORDINATOR_SYSTEM_CONFIG = "SAMZA_COORDINATOR_SYSTEM_CONFIG"
 
+  /**
+   * This environment variable is used to pass a JSON serialized map of 
configs provided during job submission.
+   */
+  val ENV_SUBMISSION_CONFIG = "SAMZA_SUBMISSION_CONFIG"
 
 Review comment:
   Good point, yes, it will be added later. I am splitting them to make PRs 
relatively small. 

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to