Skip to content

docs: correct connection parameter reference - #519

Merged
vuanhphung merged 1 commit into
mainfrom
docs/fix-connection-parameter-reference
Sep 5, 2026
Merged

docs: correct connection parameter reference#519
vuanhphung merged 1 commit into
mainfrom
docs/fix-connection-parameter-reference

Conversation

@vuanhphung

Copy link
Copy Markdown
Collaborator

Correct CONNECTION_PARAMETERS.md to match the current public, internal, and deprecated option surfaces and backend wiring. This adds omitted session and statement parameters and fixes Thrift vs. Kernel support and defaults.

Validation: npx prettier CONNECTION_PARAMETERS.md --check; git diff --check.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Thanks for your contribution! To satisfy the DCO policy in our contributing guide every commit message must include a sign-off message. One or more of your commits is missing this message. You can reword previous commit messages with an interactive rebase (git rebase -i main).

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No issues identified by the review bot.

@vuanhphung
vuanhphung force-pushed the docs/fix-connection-parameter-reference branch from 58b117a to f19470b Compare September 4, 2026 23:09

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No issues identified by the review bot.

Signed-off-by: Vu Anh Phung <vu.phung@databricks.com>
@vuanhphung
vuanhphung force-pushed the docs/fix-connection-parameter-reference branch from f19470b to 6cf98c0 Compare September 5, 2026 00:54

@peco-review-bot peco-review-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ No issues identified by the review bot.

@vuanhphung
vuanhphung added this pull request to the merge queue Sep 5, 2026
Merged via the queue into main with commit d849d1f Sep 5, 2026
24 of 25 checks passed
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.

1 participant