amindadgar commented on code in PR #38066:
URL: https://github.com/apache/airflow/pull/38066#discussion_r1525821374
##########
airflow/settings.py:
##########
@@ -273,7 +273,7 @@ def configure_orm(disable_connection_pool=False,
pool_class=None):
DEFAULT_ENGINE_ARGS = {
"postgresql": {
- "executemany_mode": "values",
+ "executemany_mode": "values_only",
Review Comment:
> Ahhh... I miss this change. Yeah we definitely should keep support of SA
1.4 as much as possible, not all providers could work with SA 2.0
>
> But also I'm not sure that we need to add support SA 2.0, because I'm not
totally sure that Airflow could work with it properly.
Agreed. The FAB library you mentioned earlier is required to update their
SA. I'll have a quick look on what is needed to update that dependency.
--
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]
