You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by lh...@apache.org on 2020/03/12 00:19:35 UTC

[samza] branch master updated: Fix the issue that getAppMainArgs and getAppMainClass is reversed (#1313)

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 <ke...@icloud.com>
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 <kw...@linkedin.com>
---
 .../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));
   }
 
 }