[SERVER-63774] Pass optional dbVersion to runPipelineOnSpecificShard Created: 17/Feb/22  Updated: 29/Oct/23  Resolved: 03/Mar/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 6.0.0-rc0, 4.4.18, 4.2.24, 5.0.13

Type: Bug Priority: Major - P3
Reporter: Jennifer Peshansky (Inactive) Assignee: Jennifer Peshansky (Inactive)
Resolution: Fixed Votes: 0
Labels: pm-2419-commit
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
backported by SERVER-69002 [5.0] backport pm-2419 Per Shard Curs... Closed
backported by SERVER-69003 [4.4] backport pm-2419 Per Shard Curs... Closed
backported by SERVER-69004 [4.2] backport pm-2419 Per Shard Curs... Closed
Depends
is depended on by SERVER-63771 Fix all temporarily blocked tests in ... Closed
Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: QE 2022-03-07
Participants:

 Description   

Error: cm uninitialized

 

variant file log run
ubuntu ASAN AFF does_not_implicitly_create_database.js logs 3
ubuntu UBSAN AFF show_expanded_events.js logs 6
ubuntu UBSAN AFF whole_db_resumability.js logs 3
ubuntu UBSAN AFF whole_db.js logs 2
ubuntu ASAN AFF whole_db_metadata_notifications.js logs 7
ubuntu ASAN AFF ddl_create_event.js logs 8


 Comments   
Comment by Githook User [ 08/Nov/22 ]

Author:

{'name': 'Mickey. J Winters', 'email': 'mickey.winters@mongodb.com', 'username': 'mjrb'}

Message: SERVER-69004 backport per shard cursors

SERVER-62400
SERVER-62681
SERVER-62738
SERVER-63781
SERVER-63774
SERVER-63772
SERVER-63773
SERVER-58673
SERVER-70633
SERVER-69785

both modified: src/mongo/s/query/cluster_aggregate.cpp
both modified: src/mongo/s/query/cluster_aggregation_planner.cpp
both modified: src/mongo/s/query/cluster_aggregation_planner.h
these files had irreconcilable differences and the changes had to be reimplemented by hand.
this is due to some refactoring and AggregationTargeter not existing

(cherry-picked from commit e424f3ed1266bcd772c52bcf23c518b2fe6c83ab)
(cherry-picked from commit ba8f9a28c8769dfe10f73b190c943ee4a57ee7a1)
(cherry-picked from commit 1219ff764c932f16a68c7ba1afd9b925f9f876c8)
Branch: v4.2
https://github.com/mongodb/mongo/commit/792f237975470ab61e3e9cd4b54ede87ed9377d8

Comment by Githook User [ 31/Oct/22 ]

Author:

{'name': 'Mickey. J Winters', 'email': 'mickey.winters@mongodb.com', 'username': 'mjrb'}

Message: SERVER-69003 backport per shard cursors

SERVER-62400
SERVER-62681
SERVER-62738
SERVER-63781
SERVER-63774
SERVER-63772
SERVER-63773
SERVER-58673
SERVER-70633
SERVER-69785

(cherry-picked from commit 4d5451b20fecaf4a054cca94de6962fcf28c577f)
(cherry-picked from commit ba8f9a28c8769dfe10f73b190c943ee4a57ee7a1)
(cherry-picked from commit 1219ff764c932f16a68c7ba1afd9b925f9f876c8)
Branch: v4.4
https://github.com/mongodb/mongo/commit/e424f3ed1266bcd772c52bcf23c518b2fe6c83ab

Comment by Githook User [ 07/Sep/22 ]

Author:

{'name': 'Mickey. J Winters', 'email': 'mickey.winters@mongodb.com', 'username': 'mjrb'}

Message: SERVER-69002 backport per shard cursors

SERVER-62400
SERVER-62681
SERVER-62738
SERVER-63781
SERVER-63774
SERVER-63772
SERVER-63773
SERVER-58673

(cherry-picked from commit fc54ebd0137a25ea664c022b51b685667dd037c7)
(cherry-picked from commit 53d7bceee61f73a1d6959edb5d490c3b338f3c0d)
(cherry-picked from commit 586663fec7c3a7d4a8b0185ff24825bd15e80dff)
(cherry-picked from commit ef2a62dcc27461d2be1b619c75bc04effa1f2021)
(cherry-picked from commit 4f3626ff4486e672569699dfde1cc0ae8c54d348)
(cherry-picked from commit 0f7683455bc06b153f14368a3f05f0b69671717e)
(cherry-picked from commit 11d01816f743d6764c4f12c42697f5edf813ce27)
(cherry-picked from commit 1fe77b5bd9fb13f9eb74275359dcc4ba69f2d5e9)
Branch: v5.0
https://github.com/mongodb/mongo/commit/4d5451b20fecaf4a054cca94de6962fcf28c577f

Comment by Githook User [ 03/Mar/22 ]

Author:

{'name': 'Jennifer Peshansky', 'email': 'jennifer.peshansky@mongodb.com', 'username': 'jenniferpeshansky'}

Message: SERVER-63774 Pass optional dbVersion to runPipelineOnSpecificShard
Branch: master
https://github.com/mongodb/mongo/commit/4f3626ff4486e672569699dfde1cc0ae8c54d348

Comment by Jennifer Peshansky (Inactive) [ 28/Feb/22 ]

The cleanest solution here seems to be removing the logic for adding versioning information from this function, and instead assume the versioning information has already been added. The caller functions can add it as necessary. That way, we avoid passing an optional chunkManager or an optional dbVersion. However, we first have to investigate how invasive it would be to pass this responsibility onto the callers of this function.

Comment by Mickey Winters [ 24/Feb/22 ]

I symbolized a stack trace really quick to make sure another test was the same issue. it seems like in sometimes we try to pass a chunk manager to runPipelineOnSpecificShardOnly in cluster_aggregate which causes the invariant trying to get an uninitialized optional. we only actually use the chunk manager in the case that we are not doing a per shard cursor, so maybe we can just make it an optional and runPipelineOnPrimaryShard can pass it in for its purposes. If this is appropriate this would be a quick fix

Generated at Thu Feb 08 05:58:38 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.