[SERVER-38324] Add validator in TransactionCoordinatorDocument IDL that the string 'decision' field is 'abort' or 'commit' Created: 29/Nov/18 Updated: 29/Oct/23 Resolved: 30/Jan/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.1.8 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Esha Maharishi (Inactive) | Assignee: | Esha Maharishi (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | ShardedTxn:DistributedCommit, neweng | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||
| Sprint: | Sharding 2019-01-28, Sharding 2019-02-11 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Description |
|
Or make the field a C++ type, perhaps txn::CoordinatorCommitDecision |
| Comments |
| Comment by Githook User [ 30/Jan/19 ] |
|
Author: {'email': 'kimberly.tao@mongodb.com', 'name': 'Kim Tao'}Message: |
| Comment by Esha Maharishi (Inactive) [ 16/Jan/19 ] |
|
kimberly.tao, ok, awesome. Let me know if you need help with it. |
| Comment by Kim Tao [ 15/Jan/19 ] |
|
I haven't yet made the change, so I can implement your idea. |
| Comment by Esha Maharishi (Inactive) [ 15/Jan/19 ] |
|
kimberly.tao, have you started working on this? Perhaps we should go the "make the field a C++ type" route, so that we can add a validator that if the decision is 'commit', then a commitTimestamp is present? |