Rename parameter `wait_policy` to `wait_for_completion` in `EmrCreateJobFlowOperator`

[vincbeck]

Apache Airflow Provider(s)

amazon

Versions of Apache Airflow Providers

Main

Apache Airflow version

Main

Operating System

Macos

Deployment

Other

Deployment details

No response

What happened

See comment.

The operator EmrCreateJobFlowOperator should follow the same standards as the other operator. All operators in Amazon provider package use the parameter wait_for_completion to configure whether the user wants to wait. However, in EmrCreateJobFlowOperator, this parameter is named wait_policy and is not a boolean but an enum.

What you think should happen instead

• Deprecate the parameter wait_policy
• Add a new parameter wait_for_completion
• Handle both parameters so that it is a non breaking change

How to reproduce

N/A

Anything else

No response

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[ethanmypan]

Hi @vincbeck, thanks for opening this issue! I’m new to open source and would love to start contributing. This looks like a good first issue, and I’d be happy to work on it if that’s okay. Could you confirm if it’s available for me to take on? Thanks!

[laksh-krishna-sharma]

@vincbeck Please review the changes and let me know if I need to make any adjustments

[vincbeck]

I just reviewed it! Thanks