fix: type annotation for num_of_dpus in GlueJobHook

[wonhyeongseo]

closes: #29091

Simple fix of type annotation for num_of_dpus according to AWS Glue Documentation

@Taragolis may you please review this PR?

[Taragolis]

You also need to change in Operator (potentially in other places)

airflow/airflow/providers/amazon/aws/operators/glue.py

Line 85 in 2e90c06

num_of_dpus: int | None = None,

I still don't know what is better annotation for this case float | None or int | float | None

[Taragolis]

Yeah, statics check failed:

airflow/providers/amazon/aws/hooks/glue.py:97: error: Incompatible types in
assignment (expression has type "float", variable has type "int")  [assignment]
                self.num_of_dpus = num_of_dpus

[wonhyeongseo]

Ok @Taragolis. Adding in float instead of replacing int for backward compatability. I checked the files for num_of_dpus, and the modified two files are all there is. May you please run the workflow again? Thanks!

[Taragolis]

pending approval (wait for CI)

[Taragolis]

🤣 mypy became crazy when tried to make static check against condition in Hook

https://github.com/apache/airflow/blob/c4c14c07a99b1554346a426aa947e68eb52ce31e/airflow/providers/amazon/aws/hooks/glue.py#L95

Add annotation here, something like: self.num_of_dpus: int | float = 10

[wonhyeongseo]

Updated as you suggested, @Taragolis! Thank you so much for guiding me. I hope to contribute more in the future. Have a great day/afternoon!

[boring-cyborg]

Awesome work, congrats on your first merged pull request!