iainjames88 commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator URL: https://github.com/apache/airflow/pull/5914#issuecomment-526130969 Thanks for the feedback :) I wonder how you feel about validating the response from the Lambda function (i.e., not just the response from the Lambda API itself) in this operator. Right now users pass a function to the constructor which is applied to the response payload to validate it. But I I'm not so sure if that's a great idea - it puts extra responsibility on to the operator when it should probably just be concerned with executing Lambda functions. Should we instead be using a sensor to inspect the response payload? Thanks again all for the input. Apologies for the changes in direction -I'm going to hang off from any other changes for now :)
---------------------------------------------------------------- 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
