-
Type:
Improvement
-
Resolution: Unresolved
-
Priority:
Major - P3
-
None
-
Affects Version/s: None
-
Component/s: None
-
Catalog and Routing
-
1
-
🟥 DDL
-
None
-
None
-
None
-
None
-
None
-
None
SERVER-120986 introduces OperationSessionTracker which increments the tracked session via getNextSession(). However, ShardingDDLCoordinator and ConfigsvrCoordinator refer to this as getNewSession(). We should consistently use either "new" or "next" to refer to the next session, with my preference being "next."
Additionally, ShardingDDLCoordinator names this function without an underscore, while ConfigsvrCoordinator names it with an underscore. Given ShardingDDLCoordinator uses underscores elsewhere, I'd suggest prefixing all of the session related methods with an underscore as well.
- depends on
-
SERVER-120986 Extract OperationSessionInfo logic into shared coordinator utility
-
- Closed
-
- is related to
-
SERVER-120986 Extract OperationSessionInfo logic into shared coordinator utility
-
- Closed
-