Skip to content

docs(jdbc): add adapter ha support#35342

Open
sheyanjie-qq wants to merge 4 commits into
mainfrom
docs/jdbc-adapter-ha-2
Open

docs(jdbc): add adapter ha support#35342
sheyanjie-qq wants to merge 4 commits into
mainfrom
docs/jdbc-adapter-ha-2

Conversation

@sheyanjie-qq
Copy link
Copy Markdown
Contributor

@sheyanjie-qq sheyanjie-qq commented May 15, 2026

Description

add adapter ha support in websocket connection

Issue(s)

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

Copilot AI review requested due to automatic review settings May 15, 2026 12:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Java connector documentation to describe the new adapterHA property for WebSocket connections, covering adapter high availability and load balancing requirements.

Changes:

  • Adds TSDBDriver.PROPERTY_KEY_ADAPTER_HA to the English Java connector properties list.
  • Adds the same adapterHA documentation to the Chinese Java connector reference.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
docs/en/14-reference/05-connector/14-java.md Documents the WebSocket-only adapterHA configuration option and version requirement.
docs/zh/14-reference/05-connector/14-java.mdx Adds the corresponding Chinese documentation for adapterHA.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/en/14-reference/05-connector/14-java.md Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 15, 2026 12:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds documentation for the adapterHA configuration property in the Java connector reference for both English and Chinese versions. This property enables high availability for adapter endpoints when using WebSocket connections. Review feedback focuses on improving the phrasing and consistency of the documentation, including suggestions for more natural English terminology and more formal Chinese wording.

Comment thread docs/en/14-reference/05-connector/14-java.md Outdated
Comment thread docs/zh/14-reference/05-connector/14-java.mdx Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 15, 2026 12:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings May 15, 2026 12:39
@sheyanjie-qq sheyanjie-qq force-pushed the docs/jdbc-adapter-ha-2 branch from c6e8b3f to ef9c98c Compare May 15, 2026 12:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants