[GitHub] [airflow] tnyz commented on pull request #34654: Remove required emr serverless operator param

["tnyz (via GitHub)" <gi...@apache.org>]

tnyz commented on PR #34654:
URL: https://github.com/apache/airflow/pull/34654#issuecomment-1738107974

   > Could you add a bit more details about this best practices? Unfortunately I can’t find any info, according to the [boto3 doc](https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/emr-serverless/client/create_application.html) this fields are mandatory, in some cases it doesn’t mean that it actually mandatory 🤣
   
   ok I think I understand what you meant. yes it seems like they are mandatory therefore they are part of operator param. this is definitely an interesting approach as I always thought these are supposed to be validated on the server side not by airflow operator api


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org