Skip to content
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

Add index correction document #2217

Merged
merged 4 commits into from Oct 27, 2023
Merged

Add index correction document #2217

merged 4 commits into from Oct 27, 2023

Conversation

ykadowak
Copy link
Contributor

Description:

SSIA

Related Issue:

Versions:

  • Go Version: 1.21.3
  • Docker Version: 20.10.8
  • Kubernetes Version: v1.28.2
  • NGT Version: 2.1.3

Checklist:

Special notes for your reviewer:

@vdaas-ci
Copy link
Collaborator

[CHATOPS:HELP] ChatOps commands.

  • 🙆‍♀️ /approve - approve
  • 🍱 /format - format codes and add licenses
  • /gen-test - generate test codes
  • 🏷️ /label - add labels
  • 🔚 2️⃣ 🔚 /label actions/e2e-deploy - run E2E deploy & integration test

This commit fixes the style issues introduced in ffe071a according to the output
from Gofumpt and Prettier.

Details: #2217
@ykadowak ykadowak requested a review from a team October 23, 2023 07:45
@ykadowak ykadowak removed the request for review from a team October 23, 2023 07:45
@github-actions github-actions bot added the type/feature New feature label Oct 23, 2023
@ykadowak ykadowak requested review from kpango and hlts2 October 23, 2023 07:45
@cloudflare-pages
Copy link

cloudflare-pages bot commented Oct 23, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: e9f5ecd
Status: ✅  Deploy successful!
Preview URL: https://2c4aeafe.vald.pages.dev
Branch Preview URL: https://feature-docs-index-correctio.vald.pages.dev

View logs

Co-authored-by: Hiroto Funakoshi <hiroto.funakoshi.hiroto@gmail.com>
@ykadowak ykadowak requested a review from hlts2 October 25, 2023 09:16
hlts2
hlts2 previously approved these changes Oct 26, 2023
Copy link
Contributor

@hlts2 hlts2 left a comment

Choose a reason for hiding this comment

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

LGTM!

## Important Notes

- Processing time
Under conditions of 10 million vectors and agent replica \*10, it takes about 10~20 minutes. The process is O(MN) where M is the number of vector items and N is the number of agent replicas.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think O(MN) is worst case.
It would be better to describe about Index Replica.

Copy link
Collaborator

@kpango kpango left a comment

Choose a reason for hiding this comment

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

LGTM

@ykadowak ykadowak merged commit d26b317 into main Oct 27, 2023
20 checks passed
@ykadowak ykadowak deleted the feature/docs/index-correction branch October 27, 2023 05:02
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
ykadowak added a commit that referenced this pull request Nov 30, 2023
@vankichi vankichi mentioned this pull request Dec 4, 2023
kmrmt pushed a commit that referenced this pull request Dec 12, 2023
* Add index correction document

* style: format code with Gofumpt and Prettier

This commit fixes the style issues introduced in ffe071a according to the output
from Gofumpt and Prettier.

Details: #2217

* Update docs/user-guides/index-correction.md

Co-authored-by: Hiroto Funakoshi <hiroto.funakoshi.hiroto@gmail.com>

* Update notes on processing time

---------

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Hiroto Funakoshi <hiroto.funakoshi.hiroto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants