[SERVER-82152] Remove CollectionCatalog usage in BucketCatalog for timeseries stats Created: 12/Oct/23  Updated: 10/Nov/23  Resolved: 10/Nov/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: Gregory Wlodarek Assignee: Gregory Wlodarek
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Storage Execution
Backwards Compatibility: Fully Compatible
Sprint: Execution Team 2023-11-13
Participants:

 Description   

Discussion from Slack:

I do agree fetching the collection at that layer feels off. I think we have two potential options in the short term:

  • Pass the Collection& into all these top-level methods and thread it through, rather than the opCtx.
  • Keep some pointer to the relevant collection stats alongside the ExecutionStats for the collection, which I think would only require threading the collection instance through a few top-level calls (maybe just tryInsert/insert?) to be able to initialize the pointer the first time we deal with a given namespace. (Preferred implementation)


 Comments   
Comment by Githook User [ 10/Nov/23 ]

Author:

{'name': 'Gregory Wlodarek', 'email': 'gregory.wlodarek@mongodb.com', 'username': 'GWlodarek'}

Message: SERVER-82152 Remove CollectionCatalog usage in BucketCatalog for time-series stats
Branch: master
https://github.com/mongodb/mongo/commit/e5754f85a309eba59c8be737e7daa4052cf9ba22

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