[SERVER-75512] Store a hashed telemetry key rather than a BSONObj Created: 30/Mar/23 Updated: 07/Feb/24 Resolved: 15/May/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 7.1.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Charlie Swanson | Assignee: | Davis Haupt (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | auto-reverted | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Assigned Teams: |
Query Optimization
|
||||||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||||||
| Sprint: | QO 2023-05-15 | ||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||
| Linked BF Score: | 162 | ||||||||||||||||||||||||||||||||
| Description |
|
The design doc says we will be hashing it, and it is important to do this to understand the releasable performance numbers. It should reduce contention on the partitioned telemetry store mutex since comparisons will be simpler and probably faster, although it is a tradeoff since we are then adding a hash function invocation to the "hot path" at query time. |
| Comments |
| Comment by Githook User [ 07/Feb/24 ] |
|
Author: {'name': 'Erin Zhu', 'email': 'erin.zhu@mongodb.com', 'username': 'erinzhu001'}Message: Includes backports of the following |
| Comment by Githook User [ 15/May/23 ] |
|
Author: {'name': 'Davis Haupt', 'email': 'davis.haupt@mongodb.com', 'username': 'davish'}Message: |
| Comment by xgen-buildbaron-user [ 10/May/23 ] |
|
Ticket re-opened due to revert. run_unittests began a consistent failure of build\install\bin\db_query_test.exe |
| Comment by Githook User [ 10/May/23 ] |
|
Author: {'name': 'auto-revert-processor', 'email': 'dev-prod-dag@mongodb.com', 'username': ''}Message: Revert " This reverts commit b6eab7e7d83b10cef5c57f781c1751b465aade9f. |
| Comment by Githook User [ 09/May/23 ] |
|
Author: {'name': 'Davis Haupt', 'email': 'davis.haupt@mongodb.com', 'username': 'davish'}Message: |
| Comment by Davis Haupt (Inactive) [ 08/May/23 ] |
|
This ticket removes cmdObj from the TelemetryEntry, and instead relies fully on the request shapifier. Since the shapifier is not yet implemented for agg, tests relying on the key for agg telemetry are failing. |