This is an automated email from the ASF dual-hosted git repository.

lhaiesp pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/samza.git


The following commit(s) were added to refs/heads/master by this push:
     new 48fc94a  Fix the issue that getAppMainArgs and getAppMainClass is 
reversed (#1313)
48fc94a is described below

commit 48fc94ae3a0fd7bad36edbe8cfc4d581964395f8
Author: Ke Wu <[email protected]>
AuthorDate: Wed Mar 11 17:19:23 2020 -0700

    Fix the issue that getAppMainArgs and getAppMainClass is reversed (#1313)
    
    Fix the issue that getAppMainArgs and getAppMainClass is reversed in 
ApplicationConfig.java
    
    Co-authored-by: Ke Wu <[email protected]>
---
 .../src/main/java/org/apache/samza/config/ApplicationConfig.java      | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java 
b/samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java
index 0854c04..7fda7e1 100644
--- a/samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java
+++ b/samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java
@@ -101,11 +101,11 @@ public class ApplicationConfig extends MapConfig {
   }
 
   public Optional<String> getAppMainArgs() {
-    return Optional.ofNullable(get(APP_MAIN_CLASS));
+    return Optional.ofNullable(get(APP_MAIN_ARGS));
   }
 
   public Optional<String> getAppMainClass() {
-    return Optional.ofNullable(get(APP_MAIN_ARGS));
+    return Optional.ofNullable(get(APP_MAIN_CLASS));
   }
 
 }

Reply via email to