fix(emr-serverless): emr serverless timeout

fix emr serverless timeout issue introduced in #43
awslabs · May 10, 2024 · 6295462 · 6295462
2 parents 57d9420 + e48def2
commit 6295462
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/src/emr_cli/deployments/emr_serverless.py b/src/emr_cli/deployments/emr_serverless.py
@@ -270,9 +270,6 @@ def run_job(
         if s3_logs_uri:
             config_overrides = {"monitoringConfiguration": {"s3MonitoringConfiguration": {"logUri": s3_logs_uri}}}
 
-        if timeout is None:
-            timeout = 12 * 60  # set to AWS default value (12 hours in minutes)
-
         response = self.client.start_job_run(
             applicationId=self.application_id,
             executionRoleArn=self.job_role,

diff --git a/src/emr_cli/emr_cli.py b/src/emr_cli/emr_cli.py
@@ -217,7 +217,7 @@ def deploy(project, entry_point, s3_code_uri):
 @click.option(
     "--emr-serverless-timeout",
     help="EMR Serverless job timeout in minutes - defaults to 12 hours",
-    default=None,
+    default=720, # set to AWS default value (12 hours in minutes)
     type=int
 )
 @click.pass_obj