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 benchmark on collection load time #204

Merged
merged 20 commits into from
Sep 27, 2024
Merged

Conversation

tellet-q
Copy link
Contributor

  • Introduce BENCHMARK_STRATEGY env variable to control the experiments, possible values: default | tenants | collection-reload
  • Add a new server config qdrant-continuous-benchmarks-snapshot with QDRANT_NUM_CPUS: 4 to ensure no optimizations are triggered after snapshot recovery
  • Add a new column collection_load_time_ms into database. Fetch the value from /telemetry response.

Prepare snapshots for cases, upload them to GCP:

  • 500k points with all possible payloads, 768x, all default values
  • 500k points with all possible payloads, 768x, on_disk:true (all possible on_disk places)
  • 500k points with all possible payloads, sparse vectors, all default values
  • 500k points with all possible payloads, sparse vectors, on_disk:true (all possible on_disk places)

@tellet-q tellet-q changed the title WIP: Add benchmark on collection load time Add benchmark on collection load time Sep 25, 2024
@tellet-q
Copy link
Contributor Author

tellet-q commented Sep 25, 2024

@tellet-q
Copy link
Contributor Author

tellet-q commented Sep 26, 2024

@KShivendu @generall I wanted to run the runLoadTimeBenchmark job in parallel, almost everything is ready in code for that, but it not possible since the 2nd server has ARM architecture and we don't have Docker images for ARM.

@tellet-q
Copy link
Contributor Author

I updated the code to run the benchmark in parallel with tenants benchmark. The benchmark runs in a separate server machine and only the client machine is shared.

https://github.com/qdrant/vector-db-benchmark/actions/runs/11051637232

Copy link
Member

@KShivendu KShivendu left a comment

Choose a reason for hiding this comment

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

LGTM. I saw that you updated the dashboard as well. Great!

@tellet-q tellet-q merged commit 5cea6f1 into master Sep 27, 2024
7 checks passed
@tellet-q tellet-q deleted the feat/load-time-benchmark branch September 27, 2024 11:22
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