[SERVER-72111] Avoid additional read from DurableCatalog in CollectionImpl::initFromExisting Created: 14/Dec/22  Updated: 29/Oct/23  Resolved: 03/Feb/23

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

Type: Improvement Priority: Major - P3
Reporter: Matthew Saltz (Inactive) 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-02-06
Participants:

 Description   

When initializing a collection object, we sometimes need to get the index ident from the durable catalog.

However, this read from the durable catalog is redundant, since we've already read the catalog entry from the durable catalog before calling this function.

We should find a way to remove the additional read from the durable catalog during CollectionImpl::initFromExisting by passing the necessary information from the durable catalog through the call site.

The current BSONCollectionCatalogEntry object, however does not contain any information about index idents, so we may need to add a way to obtain both the catalog entry and the idxIdent field with one DurableCatalog function call.

See the catalog arch guide for an example of the durable catalog format



 Comments   
Comment by Githook User [ 03/Feb/23 ]

Author:

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

Message: SERVER-72111 Avoid additional read from DurableCatalog in CollectionImpl::initFromExisting
Branch: master
https://github.com/mongodb/mongo/commit/ff06d8d8c15e6b4f79231a385539741bb169dac1

Comment by Githook User [ 03/Feb/23 ]

Author:

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

Message: SERVER-72111 Add index idents to DurableCatalogEntry
Branch: master
https://github.com/mongodb/mongo/commit/b2985ae30e7f4ff6a80690ef9d70d8215b353892

Comment by Henrik Edin [ 25/Jan/23 ]

I think this ticket should include using ident when instantiating the index catalog over just doing everything via indexName.

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