[CSHARP-2544] Move pinned server from Session to CoreTransaction Created: 13/Mar/19 Updated: 28/Oct/23 Resolved: 18/Apr/19 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | Transactions |
| Affects Version/s: | None |
| Fix Version/s: | 2.9.0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Vincent Kam (Inactive) | Assignee: | Vincent Kam (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Epic Link: | CSHARP 4.2 Transactions Support | ||||||||
| Description |
|
Currently in 2.9.0-beta1, the pinned server (for mongos pinning) is stored in as a public property in the session object. It may make more sense to store it within the CoreTransaction object because a new CoreTransaction is generated for each transaction, saving us the trouble of resetting the pinned server in the session to null. If we make this change, it would have to before the final release of 2.9.0 as it would otherwise be a breaking change to the ICoreSession api. |