Fix Impala configuration argument construction

aws · Feb 18, 2016 · 8ca6ea8 · 8ca6ea8
1 parent d8fba3b
commit 8ca6ea8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/awscli/customizations/emr/applicationutils.py b/awscli/customizations/emr/applicationutils.py
@@ -114,7 +114,7 @@ def _build_impala_install_bootstrap_action(region, args=None):
         constants.LATEST]
     if args is not None:
         args_list.append(constants.IMPALA_CONF)
-        args_list += args
+        args_list.append(','.join(args))
     return emrutils.build_bootstrap_action(
         name=constants.INSTALL_IMPALA_NAME,
         path=emrutils.build_s3_link(

diff --git a/tests/unit/customizations/emr/test_create_cluster_ami_version.py b/tests/unit/customizations/emr/test_create_cluster_ami_version.py
@@ -892,7 +892,7 @@ def test_install_impala_with_all_fields(self):
         result = copy.deepcopy(DEFAULT_RESULT)
         ba = copy.deepcopy(INSTALL_IMPALA_BA)
         ba['ScriptBootstrapAction']['Args'] += \
-            ['--impala-conf', 'arg1', 'arg2']
+            ['--impala-conf', 'arg1,arg2']
         result['BootstrapActions'] = [ba]
         self.assert_params_for_cmd(cmd, result)
 
@@ -952,7 +952,7 @@ def test_applications_all_types_from_json_file(self):
         impala_ba = copy.deepcopy(INSTALL_IMPALA_BA)
         impala_ba['ScriptBootstrapAction']['Args'] += \
             ['--impala-conf',
-             'IMPALA_BACKEND_PORT=22001', 'IMPALA_MEM_LIMIT=70%']
+             'IMPALA_BACKEND_PORT=22001,IMPALA_MEM_LIMIT=70%']
         ba_list = [INSTALL_GANGLIA_BA, INSTALL_HBASE_BA,
                    impala_ba]
         step_list = [INSTALL_HIVE_STEP, INSTALL_PIG_STEP, INSTALL_HBASE_STEP]