Skip to content

[#6390] skip serializing shard details in Level2#6398

Merged
generall merged 2 commits intotelemetry-levelsfrom
skip-serializing-shards-in-level2
Apr 17, 2025
Merged

[#6390] skip serializing shard details in Level2#6398
generall merged 2 commits intotelemetry-levelsfrom
skip-serializing-shards-in-level2

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