You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "dimonchik-suvorov (via GitHub)" <gi...@apache.org> on 2023/06/14 13:06:58 UTC
[GitHub] [airflow] dimonchik-suvorov opened a new pull request, #31899: Making Databricks run related multi-query string in one session again…
dimonchik-suvorov opened a new pull request, #31899:
URL: https://github.com/apache/airflow/pull/31899
… (#31898)
What I'm doing here is simply reverting the refactoring that was done. Unfortunately, I didn't find the proper way to test this situation because we are mocking `get_conn` in tests which is a big deal and contains a lot of logic...
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #31899: Making Databricks run related multi-query string in one session again
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #31899:
URL: https://github.com/apache/airflow/pull/31899#issuecomment-1591930399
LGTM.
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] dimonchik-suvorov commented on pull request #31899: Making Databricks run related multi-query string in one session again
Posted by "dimonchik-suvorov (via GitHub)" <gi...@apache.org>.
dimonchik-suvorov commented on PR #31899:
URL: https://github.com/apache/airflow/pull/31899#issuecomment-1591510866
@kazanzhy as introduser of the refactoring
@potiuk as guru :)
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] potiuk merged pull request #31899: Making Databricks run related multi-query string in one session again
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #31899:
URL: https://github.com/apache/airflow/pull/31899
--
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: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org