Skip to content

Conversation

coszio
Copy link
Contributor

@coszio coszio commented Apr 17, 2025

This makes the collection telemetry to not include "shards", "transfers", and "resharding" fields when details level is < 3

@coszio coszio requested a review from generall April 17, 2025 16:50
@coszio coszio mentioned this pull request Apr 17, 2025
@generall generall merged commit 839fb04 into telemetry-levels Apr 17, 2025
16 checks passed
@generall generall deleted the skip-serializing-shards-in-level2 branch April 17, 2025 21:01
generall added a commit that referenced this pull request Apr 17, 2025
* allow shard states in anonymize telemetry (with hashed peer ids)

* introduce level3 and level4 for telemetry, where level4 = everything, incl. segments info

* upd openapi

* fix tests

* expose vector count & size stats on shard level to avoid reading of segments

* fix spelling

* upd schema

* fix tests

* Use unwrap_or_default

* [#6390] skip serializing shard details in Level2 (#6398)

* skip serializing shard details in Level2

* upd openapi

---------

Co-authored-by: generall <andrey@vasnetsov.com>

---------

Co-authored-by: Tim Visée <tim+github@visee.me>
Co-authored-by: Luis Cossío <luis.cossio@qdrant.com>
pull bot pushed a commit to kp-forks/qdrant that referenced this pull request Apr 21, 2025
* allow shard states in anonymize telemetry (with hashed peer ids)

* introduce level3 and level4 for telemetry, where level4 = everything, incl. segments info

* upd openapi

* fix tests

* expose vector count & size stats on shard level to avoid reading of segments

* fix spelling

* upd schema

* fix tests

* Use unwrap_or_default

* [qdrant#6390] skip serializing shard details in Level2 (qdrant#6398)

* skip serializing shard details in Level2

* upd openapi

---------

Co-authored-by: generall <andrey@vasnetsov.com>

---------

Co-authored-by: Tim Visée <tim+github@visee.me>
Co-authored-by: Luis Cossío <luis.cossio@qdrant.com>
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