[samza] branch master updated: Making shell environment variable case sensitive as samza configs are case sensitive. (#1100)

[atoomula]

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 <atoom...@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;