[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: |
| Comment by Githook User [ 03/Feb/23 ] |
|
Author: {'name': 'Gregory Wlodarek', 'email': 'gregory.wlodarek@mongodb.com', 'username': 'GWlodarek'}Message: |
| 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. |