Skip to content

fix(csharp): update hiveserver2 submodule to fix HTTP path query encoding#341

Merged
eric-wang-1990 merged 2 commits intomainfrom
fix/hiveserver2-http-path-query-encoding
Mar 12, 2026
Merged

fix(csharp): update hiveserver2 submodule to fix HTTP path query encoding#341
eric-wang-1990 merged 2 commits intomainfrom
fix/hiveserver2-http-path-query-encoding

Conversation

@eric-wang-1990
Copy link
Copy Markdown
Collaborator

Summary

  • Updates the hiveserver2 submodule to the latest main which includes the fix for UriBuilder percent-encoding ? as %3F in HTTP paths containing query parameters (e.g. /sql/1.0/warehouses/<id>?o=<orgId>)
  • This caused connection failures for Databricks warehouse paths that include an org ID suffix

Upstream fix

adbc-drivers/hiveserver2#26

Test plan

  • Verify connection succeeds with HTTP path containing ?o=<orgId> query parameter

🤖 Generated with Claude Code

eric-wang-1990 and others added 2 commits March 12, 2026 09:56
UriBuilder was percent-encoding '?' as '%3F' in HTTP paths containing
query parameters (e.g. /sql/1.0/warehouses/<id>?o=<orgId>). This caused
connection failures for warehouse paths with org ID suffixes.

Upstream fix: adbc-drivers/hiveserver2#26

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@eric-wang-1990 eric-wang-1990 changed the title fix: update hiveserver2 submodule to fix HTTP path query encoding fix(csharp): update hiveserver2 submodule to fix HTTP path query encoding Mar 12, 2026
@eric-wang-1990 eric-wang-1990 added this pull request to the merge queue Mar 12, 2026
@eric-wang-1990 eric-wang-1990 removed this pull request from the merge queue due to a manual request Mar 12, 2026
@eric-wang-1990 eric-wang-1990 added this pull request to the merge queue Mar 12, 2026
Merged via the queue into main with commit b716671 Mar 12, 2026
17 of 18 checks passed
@eric-wang-1990 eric-wang-1990 deleted the fix/hiveserver2-http-path-query-encoding branch March 12, 2026 17:30
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