-
Type: Improvement
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
None
-
Replication
-
Minor Change
-
Repl 2024-09-30, Repl 2024-10-14, Repl 2024-10-28, Repl 2024-11-11, Repl 2024-11-25
Almost all of the usages of WriteConcernFailed today are cases where waiting for write concern timed out. It's worth considering changing this code name to better reflect that.
The only usage we found that isn't for a timeout purpose is on mongos where it is used to combine multiple write concern errors into one. That should maybe have its own error type or we should instead just return one of the errors instead of treating this case specially, in the spirit of this comment, regarding a similar situation where we return MultipleErrorsOccurred when we get multiple write errors.
I believe drivers should only be checking error codes and labels, not error code names, so I don't think this would present an issue for them.
See comment for some additional context from Slack discussion.
- is depended on by
-
COMPASS-8365 Investigate changes in SERVER-94813: Consider changing WriteConcernFailed code name to WriteConcernTimedOut
- Closed
-
TOOLS-3687 Investigate changes in SERVER-94813: Consider changing WriteConcernFailed code name to WriteConcernTimedOut
- Closed