mik-laj commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-533823603
Travis is sad. Can you fix it?
This is an automated message from the Apache
mik-laj commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-524862605
I think, Airflow operators should be synchronous so that Airflow retrieves
information about the success of the operation. The operator
mik-laj commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-524808769
One more small change.
``AWSLambdaInvokeFunctionOperator`` > ``AwsLambdaInvokeFunctionOperator``
This will allow easier name reading.
mik-laj commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-524807101
Operator suffix is required: The service name should be a prefix.We expect
operator names to be in the form of
mik-laj commented on issue #5914: [AIRFLOW-5311] Add an AWS Lambda Operator
URL: https://github.com/apache/airflow/pull/5914#issuecomment-524796942
Quick comment. The operator's name should describe the behavior, not just
the name of the service. It should contain a verb.