Lee-W commented on PR #70114: URL: https://github.com/apache/airflow/pull/70114#issuecomment-5019460624
found it https://github.com/ClickHouse/clickhouse-connect/blob/47dbad3d09996438f701b68e48ffd8dd7da133f8/clickhouse_connect/dbapi/cursor.py#L62. I think it would be better to check the second parameter as well, since it's technically controlled by Airflow. Even though we don't currently pass it to `test_connection`, we might do so in the future—or accidentally start doing so. -- 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]
