vandonr-amz commented on code in PR #32386:
URL: https://github.com/apache/airflow/pull/32386#discussion_r1254779635
##########
airflow/providers/amazon/aws/operators/rds.py:
##########
@@ -47,14 +47,14 @@ def __init__(self, *args, aws_conn_id: str = "aws_conn_id",
hook_params: dict |
if hook_params is not None:
warnings.warn(
"The parameter hook_params is deprecated and will be removed. "
+ "Note that it is also incompatible with deferrable mode. "
"If you were using it, please get in touch either on airflow
slack, "
"or by opening a github issue on the project. "
"You can mention https://github.com/apache/airflow/pull/32352";,
AirflowProviderDeprecationWarning,
stacklevel=3, # 2 is in the operator's init, 3 is in the user
code creating the operator
)
- self.hook_params = hook_params or {}
Review Comment:
I don't exactly know how that works in practice... I suppose you'd have a
"main" region where you do the heavy lifting (sagemaker stuff for instance),
but you'd have DBs in various regions, and you'd like to be able to hit those
without having to change too much stuff ?
Are you saying that the region param should be obsoleted everywhere in the
AWS provider in favor of several connection ids ?
--
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]
