SkastVnT commented on PR #66895:
URL: https://github.com/apache/airflow/pull/66895#issuecomment-4484869190
Thanks for the review — I addressed the requested changes.
Updates made:
- moved query_tags after caller in DatabricksSqlHook.__init__ to preserve
positional-call compatibility
- added warning logs when query tag values are truncated
- clarified that raw tag values are truncated before escaping
- documented that QUERY_TAGS is part of session_configuration and only
applies when a new Databricks SQL connection is opened
- made _get_airflow_query_tags use context.get("ti") and return {} when no
task instance is present
- cleaned up the PR description wording around the old connector-kwarg test
wording
- added/updated unit coverage for query tag serialization, parameter
ordering, and missing task instance context
--
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]
