AwsBaseHook make client_type & resource_type optional params for get_client_type & get_resource_type

[eladkal]

Currently user must specify client_type to for get_client_type so an example usage is:

emr_client = AwsBaseHook(self.aws_conn_id, client_type='emr', region_name='us-east-1').get_client_type('emr')

however we already have client_type as class attribute so we can simplify it to:

emr_client = AwsBaseHook(self.aws_conn_id, client_type='emr', region_name='us-east-1').get_client_type()

EDIT:
Same applied for resource_type with get_resource_type

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

[xinbinhuang]

.get_resource_type has the same problem. Would be nice to change it here as well.

It will be even better! if we can refactor them somehow, it seems really confusing to me to have client|resource_type in both the constructor and the methods. Also, the method name looks confusing to me... (well just a personal opinion)

[github-actions]

The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease.

[eladkal]

As suggested I added deprecation for client_type/resource_type in the function.
Current implementation may leads to a very confusing cases like:

emr_client = AwsBaseHook(self.aws_conn_id, client_type='emr', region_name='us-east-1').get_client_type(client_type='dynamodb')

The values should be set only from the hook

[xinbinhuang]

LGTM!