Skip to content

[QI-3367] Allow specifiying query tags as a dict upon connection creation#749

Open
jiabin-hu wants to merge 2 commits intodatabricks:mainfrom
jiabin-hu:query_tag2
Open

[QI-3367] Allow specifiying query tags as a dict upon connection creation#749
jiabin-hu wants to merge 2 commits intodatabricks:mainfrom
jiabin-hu:query_tag2

Conversation

@jiabin-hu
Copy link
Contributor

@jiabin-hu jiabin-hu commented Mar 4, 2026

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix
  • Other

Description

For session level query tags,

  1. adding the option to specify session query tags as a dict rather than a serialized string, to be consistent w/ statement level
  2. if specified both as dict param, and serialized string as session param, dict param takes precedence, overwriting everything in session param

How is this tested?

  • Unit tests
  • E2E Tests
  • Manually
  • N/A

Related Tickets & Documents

Signed-off-by: Jiabin Hu <jiabin.hu@databricks.com>
Signed-off-by: Jiabin Hu <jiabin.hu@databricks.com>
Copy link
Contributor

@sreekanth-db sreekanth-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sreekanth-db sreekanth-db requested a review from gopalldb March 5, 2026 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants