You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by at...@apache.org on 2019/07/10 18:57:22 UTC
[samza] branch master updated: Making shell environment variable
case sensitive as samza configs are case sensitive. (#1100)
This is an automated email from the ASF dual-hosted git repository.
atoomula 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 e9f7368 Making shell environment variable case sensitive as samza configs are case sensitive. (#1100)
e9f7368 is described below
commit e9f7368996213a37b4ac11992e93116fa19c6c9a
Author: Aditya Toomula <at...@linkedin.com>
AuthorDate: Wed Jul 10 11:57:17 2019 -0700
Making shell environment variable case sensitive as samza configs are case sensitive. (#1100)
---
.../src/main/java/org/apache/samza/sql/client/cli/CliEnvironment.java | 1 -
1 file changed, 1 deletion(-)
diff --git a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/cli/CliEnvironment.java b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/cli/CliEnvironment.java
index e3bf633..3cd9e55 100644
--- a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/cli/CliEnvironment.java
+++ b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/cli/CliEnvironment.java
@@ -62,7 +62,6 @@ class CliEnvironment {
* -2: invalid value
*/
int setEnvironmentVariable(String name, String value) throws ExecutorException{
- name = name.toLowerCase();
if(name.equals(CliConstants.CONFIG_EXECUTOR)) {
createShellExecutor(value);
activeExecutorClassName = value;