[SERVER-75596] Stress $queryStats read path for aggregation Created: 03/Apr/23  Updated: 05/Jan/24  Resolved: 06/Dec/23

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 7.3.0-rc0

Type: Task Priority: Major - P3
Reporter: Charlie Swanson Assignee: Mariano Shaar
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
depends on SERVER-76189 telemetry parsing accidentally ignore... Closed
depends on SERVER-73152 Literal and field name redaction for ... Closed
depends on SERVER-76220 Handle expression context issues in a... Closed
depends on SERVER-76555 Stress-test $queryStats in core passt... Closed
depends on SERVER-76357 Support query shape serialization in ... Closed
depends on SERVER-75343 Fix field name redaction Closed
Problem/Incident
Related
related to SERVER-81994 $densify range doesn't re-parse corre... Closed
related to SERVER-82004 $bucketAuto granularity doesn't re-pa... Closed
related to SERVER-82005 query_limits_test generates a represe... Closed
related to SERVER-82006 $setWindowFields window unit fails to... Closed
related to SERVER-82007 $setWindowFields window bounds fails ... Closed
related to SERVER-82008 $setWindowFields $expMovingAvg alpha ... Closed
related to SERVER-83912 Re-parsing queryStats key when readin... Closed
related to SERVER-75893 Allow js hooks to run js file with so... Closed
related to SERVER-76088 Fix sort and hint with $natural query... Closed
Assigned Teams:
Query Integration
Backwards Compatibility: Fully Compatible
Backport Requested:
v7.2
Sprint: QO 2023-04-17, QO 2023-05-01, QO 2023-05-15, QO 2023-05-29
Participants:
Linked BF Score: 175

 Description   

See this TODO: https://github.com/mongodb/mongo/blob/79410907d1ff3ad614496713e6440c5e88991eed/buildscripts/resmokeconfig/suites/query_stats_passthrough_writeonly.yml#L1

Our current query_stats_passthrough_writeonly suite runs existing tests with query stats collection enabled, but it doesn't ever run the $queryStats stage to go ask for the telemetry data. This is a gap in coverage as some interesting redaction logic goes on there, and we need to make sure we can re-parse everything we stored.



 Comments   
Comment by Githook User [ 06/Dec/23 ]

Author:

{'name': 'Mariano Shaar', 'email': 'mariano.shaar@mongodb.com', 'username': 'shaargtz'}

Message: SERVER-75596 Stress $queryStats read path for aggregation

GitOrigin-RevId: 5e6a4ccd122dd9d6d4fdeb2992430bfb46022354
Branch: master
https://github.com/mongodb/mongo/commit/f2c76a3c5e73352ecb15220eb7c5e58a659cad80

Comment by Githook User [ 09/Oct/23 ]

Author:

{'name': 'Charlie Swanson', 'email': 'charlie.swanson@mongodb.com', 'username': 'cswanson310'}

Message: Revert "SERVER-75596 Stress $queryStats read path for aggregation"
Branch: master
https://github.com/mongodb/mongo/commit/db602c22f2a48abbfab13a1f2f25e65271c4cb50

Comment by Githook User [ 06/Oct/23 ]

Author:

{'name': 'Mariano Shaar', 'email': '47160550+shaargtz@users.noreply.github.com', 'username': 'shaargtz'}

Message: SERVER-75596 Stress $queryStats read path for aggregation
Branch: master
https://github.com/mongodb/mongo/commit/47cfc1ad9a0ef17a7cff38031b1dd88ead3effcf

Comment by Charlie Swanson [ 21/Apr/23 ]

We've uncovered some dependencies while trying to enable telemetry in the agg suites. Linking them here.

Comment by Githook User [ 12/Apr/23 ]

Author:

{'name': 'Uladzimir Makouski', 'email': 'uladzimir.makouski@mongodb.com', 'username': 'umakouski'}

Message: Revert "SERVER-75596 Invoke $telemetry in the background of telemetry suite."

This reverts commit c0e7ce05c657ffef5da25aa0399737311f3d48dc.
Branch: master
https://github.com/mongodb/mongo/commit/d0600865bd4a2fccfded5d06b5c9553f8f411b11

Comment by Githook User [ 11/Apr/23 ]

Author:

{'name': 'Charlie Swanson', 'email': 'charlie.swanson@mongodb.com', 'username': 'cswanson310'}

Message: SERVER-75596 Invoke $telemetry in the background of telemetry suite.
Branch: master
https://github.com/mongodb/mongo/commit/c0e7ce05c657ffef5da25aa0399737311f3d48dc

Comment by Charlie Swanson [ 03/Apr/23 ]

Took a stab at this and hit the bug that will.buerger@mongodb.com and I talked about. Marking this as blocked on SERVER-75343. Once that lands or once I get a chance to rebase and compile again, we can see if there are other failures hiding here.

Generated at Thu Feb 08 06:30:35 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.