iainjames88 commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-524823685
 
 
   So, looking at the code again I don't think this will handle asynchronous 
calls to Lambda very well, if at all. See that 
[invocation_type](https://github.com/apache/airflow/blob/c92181226170a015813a34daecb43df3052c0cdf/airflow/contrib/hooks/aws_lambda_hook.py#L48)
 may be `RequestResponse` for a synchronous call or `Event` for an asyncrhonous 
call. Some info 
[here](https://docs.aws.amazon.com/lambda/latest/dg/lambda-invocation.html).
   
   For my use case making synchronous calls is fine so I've been using the 
default "RequestResponse" invocation type. I think I am going to remove the 
`invocation_type` constructor param and let this be a simple synchronous 
operator. I can follow up with another PR to add support for asynchronous 
calls, later.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to