kazanzhy commented on PR #23971: URL: https://github.com/apache/airflow/pull/23971#issuecomment-1184532220
If talking globally I'm a fan of backward compatibility, especially between minor versions. In this case, I would prefer to keep it also, because the method already can take a list of statements. Actually in this PR for now I don't see "breaking changes", it's just signature unification, moving commonly used methods (`strip_sql_string` and `split_sql_string`) to DbApiHook, and probably returning result unification (except Databricks) Also adding a new parameter `split_statments=False` might be a good idea and if @potiuk also supports it I will start working on that -- 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]
