GODRIVER-2962 Remove setters from AbortTransaction and CommitTransaction.

[matthewdale]

GODRIVER-2962

Summary

Remove setters from AbortTransaction and CommitTransaction.

Background & Motivation

The field setters on the operation types in the x/mongo/driver/operation package provide no usability improvement but create a maintenance burden whenever we have to update the operations. Remove the field setters and use struct literals instead.

Split the change into multiple PRs to make reviewing them easier.

[github-actions]

API Change Report

./x/mongo/driver/operation

incompatible changes

(*AbortTransaction).ClusterClock: removed
(*AbortTransaction).Collection: removed
(*AbortTransaction).CommandMonitor: removed
(*AbortTransaction).Crypt: removed
(*AbortTransaction).Database: removed
(*AbortTransaction).Deployment: removed
(*AbortTransaction).RecoveryToken: removed
(*AbortTransaction).Retry: removed
(*AbortTransaction).ServerAPI: removed
(*AbortTransaction).ServerSelector: removed
(*AbortTransaction).Session: removed
(*AbortTransaction).WriteConcern: removed
(*CommitTransaction).ClusterClock: removed
(*CommitTransaction).CommandMonitor: removed
(*CommitTransaction).Crypt: removed
(*CommitTransaction).Database: removed
(*CommitTransaction).Deployment: removed
(*CommitTransaction).MaxTime: removed
(*CommitTransaction).RecoveryToken: removed
(*CommitTransaction).Retry: removed
(*CommitTransaction).ServerAPI: removed
(*CommitTransaction).ServerSelector: removed
(*CommitTransaction).Session: removed
(*CommitTransaction).WriteConcern: removed
NewAbortTransaction: removed
NewCommitTransaction: removed

compatible changes

AbortTransaction.Clock: added
AbortTransaction.Crypt: added
AbortTransaction.Deployment: added
AbortTransaction.Monitor: added
AbortTransaction.RecoveryToken: added
AbortTransaction.Retry: added
AbortTransaction.Selector: added
AbortTransaction.ServerAPI: added
AbortTransaction.Session: added
AbortTransaction.WriteConcern: added
CommitTransaction.Clock: added
CommitTransaction.Crypt: added
CommitTransaction.Deployment: added
CommitTransaction.MaxTime: added
CommitTransaction.Monitor: added
CommitTransaction.RecoveryToken: added
CommitTransaction.Retry: added
CommitTransaction.Selector: added
CommitTransaction.ServerAPI: added
CommitTransaction.Session: added
CommitTransaction.WriteConcern: added

[blink1073]

LGTM!

The base branch was changed.

[blink1073]

Should this also be on the 2.0 epic?

[matthewdale]

Not sure what's going on with the tests, but this isn't important to complete right now. I'm closing this for now, will revisit later.

[matthewdale]

Should this also be on the 2.0 epic?

@blink1073 No, these can go in any release because they're unstable API. I switched them to master because I didn't think the code improvement was worth the additional overhead of syncing between branches. I actually closed this PR because I'm not going to get back to it soon and I didn't want to keep it open for a long time.