-
Type: Improvement
-
Resolution: Won't Do
-
Priority: Unknown
-
None
-
Affects Version/s: None
-
Component/s: None
-
None
Some fields on a description.Server are not part of a handshake response and can become stale, leading to confusion and bugs.
Fields that should be deprecated:
- AverageRTT, AverageRTTSet - The average RTT and the "set=true" fields are set after handshake to the value of the current average RTT for the server. These can be very misleading because the average RTT changes over time, but these fields are never updated after handshake. Also, not all instances of description.Server are guaranteed to have these fields set, leading to additional confusion. All driver code should fetch the current RTT values from the RTTMonitor. There is no known external use case.
- HeartbeatInterval - The heartbeat interval is set to the Client's heartbeat interval after handshake and is used in the server selection functions in the description package. There is no known external use case.
Note: We may want to keep this field around, but as an unexported field.
Definition of done:
- Deprecate AverageRTT, AverageRTTSet, and HeartbeatInterval.
- is depended on by
-
GODRIVER-2617 Remove or un-export all currently deprecated code in Go Driver 2.0
- Closed
- related to
-
GODRIVER-2694 Return errors from "description.NewServer" and "fsm.apply"
- Backlog
-
DRIVERS-2552 Remove average and 90th percentile RTT times from server description
- Closed