Skip to content

Update qdrant.py#204

Merged
m1rl0k merged 1 commit intotestfrom
bubble-bullshit
Jan 26, 2026
Merged

Update qdrant.py#204
m1rl0k merged 1 commit intotestfrom
bubble-bullshit

Conversation

@m1rl0k
Copy link
Collaborator

@m1rl0k m1rl0k commented Jan 26, 2026

No description provided.

@m1rl0k m1rl0k merged commit 1b7a2b0 into test Jan 26, 2026
1 check failed
@augmentcode
Copy link

augmentcode bot commented Jan 26, 2026

🤖 Augment PR Summary

Summary: Enables on_disk_payload when creating new Qdrant collections via the ingest script to store payload data on disk.


Technical Notes: This applies only on collection creation in _ensure_collection_with_mode (existing collections are unchanged).

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

sparse_vectors_config=sparse_cfg,
hnsw_config=models.HnswConfigDiff(m=16, ef_construct=256),
quantization_config=quant_cfg,
on_disk_payload=True, # Enable zstd compression for 50-70% storage savings
Copy link

Choose a reason for hiding this comment

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

on_disk_payload=True controls whether payload is stored on disk (primarily a RAM-usage tradeoff), but the inline comment mentions “zstd compression” and specific storage savings which doesn’t appear to be what this setting guarantees. Also note this only applies on initial collection creation in this function, so existing collections won’t pick it up unless recreated/updated.

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎

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