eladkal commented on code in PR #26764:
URL: https://github.com/apache/airflow/pull/26764#discussion_r984840776
##########
airflow/providers/snowflake/hooks/snowflake.py:
##########
@@ -41,6 +42,18 @@ def _try_to_boolean(value: Any):
return value
+def _maybe_add_prefix(val):
+ """
+ From Airflow 2.3 onward, there is no longer a need to add the
`extra__<conn type>__`
Review Comment:
The bump can happen only after 11-Oct because we might need to make ad-hoc
release to fix something urgently. Right after 11-Oct we can bump all providers
and then proceed.
The reason I suggested it is because we usually do 1 release per month so
the next release is expected around the end of October thus the current
"backport" code for 2.2 will never reach to the user anyway.
We can also merge this first and do cleanup after.. That is OK.
--
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]
