-
Notifications
You must be signed in to change notification settings - Fork 673
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update vector search docs #18779
base: master
Are you sure you want to change the base?
update vector search docs #18779
Conversation
[LGTM Timeline notifier]Timeline:
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/approve cancel |
PR needs rebase. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
||
> **Warning:** | ||
> | ||
> The vector search feature is experimental. It is not recommended that you use it in the production environment. This feature might be changed or removed without prior notice. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
> The vector search feature is experimental. It is not recommended that you use it in the production environment. This feature might be changed or removed without prior notice. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. | |
> The vector search feature is experimental and some behaviors may change in future versions. It is not recommended that you use it in the production environment. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. |
Remove "This feature might be changed or removed without prior notice." as Vector Search is experimental because of stability issues, not product decision issues. Vector Search will never be removed.
|
||
> **Warning:** | ||
> | ||
> The vector search feature is in beta. It might be changed or removed without prior notice. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
> The vector search feature is in beta. It might be changed or removed without prior notice. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. | |
> The vector search feature is in beta and some behaviors may change in future versions. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. |
Signed-off-by: JaySon-Huang <tshent@qq.com>
## From v7.x to v8.4 or a later version | ||
|
||
Starting from v8.4, the underlying storage format of TiFlash has been updated to support the [vector search](/vector-search-overview.md). Therefore, after the upgrade TiFlash to v8.4 or a later version, in-place downgrading to the original version is not supported. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI, I've pushed a commit about the tiflash upgrade notice
> | ||
> The vector search feature is experimental. It is not recommended that you use it in the production environment. This feature might be changed or removed without prior notice. If you find a bug, you can report an [issue](https://github.com/pingcap/tidb/issues) on GitHub. | ||
|
||
</CustomContent> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<CustomContent platform="tidb-cloud"> | |
|
||
> **Note:** | ||
> | ||
> Vector search index is only available for [TiDB Cloud Serverless](/tidb-cloud/select-cluster-tier.md#tidb-cloud-serverless) clusters. | ||
> The vector search feature is only available for TiDB Self-Managed clusters and [TiDB Cloud Serverless](/tidb-cloud/select-cluster-tier.md#tidb-cloud-serverless) clusters. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
</CustomContent> | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This note is supposed to be visible for both TiDB and TiDB Cloud docs.
|
||
ALTER TABLE foo ADD VECTOR INDEX idx_name ((VEC_COSINE_DISTANCE(data))) USING HNSW; | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这注意建议删掉,在特性改为 GA 时很容易把此处漏改。L90 还有一处。
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中文版是 <USERNAME>
,需要保持一致
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中文版为 <USERNAME>
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
中文版是 <USERNAME>
Co-authored-by: xixirangrang <hfxsd@hotmail.com> Co-authored-by: EricZequan <110292382+EricZequan@users.noreply.github.com>
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `<USER>`: The username to connect to the TiDB cluster. | |
- `<USERNAME>`: The username to connect to the TiDB cluster. |
@qiancai: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `<USER>`: The username to connect to the TiDB cluster. | |
- `<USERNAME>`: The username to connect to the TiDB cluster. |
|
||
- `<HOST>`: The host of the TiDB cluster. | ||
- `<PORT>`: The port of the TiDB cluster. | ||
- `<USER>`: The username to connect to the TiDB cluster. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- `<USER>`: The username to connect to the TiDB cluster. | |
- `<USERNAME>`: The username to connect to the TiDB cluster. |
First-time contributors' checklist
What is changed, added or deleted? (Required)
This PR moves 15 vector search docs from the tidb-cloud folder to the vector-search folder to so they can be reused by TiDB self-managed docs.
Which TiDB version(s) do your changes apply to? (Required)
Tips for choosing the affected version(s):
By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.
For details, see tips for choosing the affected versions.
What is the related PR or file link(s)?
Do your changes match any of the following descriptions?