Taragolis commented on PR #24306: URL: https://github.com/apache/airflow/pull/24306#issuecomment-1150009182
> Basically, emr_conn_id should not mandatory. I mean if user set `emr_conn_id="this_connection_not_exists"` this connection not exists and should raise an error rather than set to empty extras. And `emr_conn_id=None` - is not mandatory. I think it also required make changes in operator https://github.com/apache/airflow/blob/287fc4bdeb107fc81653d3a6721f9f5b87b90529/airflow/providers/amazon/aws/operators/emr.py#L304-L305 -- 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]
