[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:
Depends
depends on SERVER-38327 IDL does not support optional types w... Closed
Gantt Dependency
has to be done before SERVER-38307 Add test coverage for manual tamperin... Closed
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: SERVER-38324: add validation to TransactionCoordinatorDocument decision
Branch: master
https://github.com/mongodb/mongo/commit/6dc4072453b4ba17fed636f049c10cf1316357bb

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?

Generated at Thu Feb 08 04:48:38 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.