kakatur commented on code in PR #64745:
URL: https://github.com/apache/airflow/pull/64745#discussion_r3463530852


##########
providers/amazon/src/airflow/providers/amazon/aws/operators/batch.py:
##########
@@ -220,14 +220,20 @@ def execute(self, context: Context) -> str | None:
 
         if self.deferrable:
             if not self.job_id:
-                raise AirflowException("AWS Batch job - job_id was not found")
+                raise ValueError("AWS Batch job - job_id was not found")

Review Comment:
   Updated docstring to document both ValueError (missing job_id) and 
AirflowException (job submission/execution failures) for accuracy."



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to