isaiahiruoha commented on code in PR #53990:
URL: https://github.com/apache/airflow/pull/53990#discussion_r2246452050
##########
providers/amazon/src/airflow/providers/amazon/aws/executors/aws_lambda/lambda_executor.py:
##########
@@ -398,8 +398,8 @@ def process_queue(self, queue_url: str):
self.log.warning("Deleting the message to avoid processing it
again.")
self.sqs_client.delete_message(QueueUrl=queue_url,
ReceiptHandle=receipt_handle)
continue
- return_code = body.get("return_code")
ser_task_key = body.get("task_key")
+ return_code = body.get("return_code")
Review Comment:
True, I had originally switched the order when adding the `command`
extraction since `ser_task_key` was always needed, and then `return_code` and
`command` were based on queue type. But with no `command` in the output,
switching does not make sense. Adjusted, thanks.
--
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]
