[GODRIVER-1584] Ensure deployment-wide fields are updated in handshakes/heartbeats Created: 20/Apr/20 Updated: 28/Oct/23 Resolved: 05/May/20 |
|
| Status: | Closed |
| Project: | Go Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.4.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Divjot Arora (Inactive) | Assignee: | Divjot Arora (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
The $clusterTime is not updated in handshakes/heartbeats because there is no ClusterClock passed down to the topology layer. I think this bug was hidden because the testing library uses the test's client to create the initial collection, so the create command updates the cluster time. We should also fix this by using the global client for creating/dropping collections. |
| Comments |
| Comment by Githook User [ 05/May/20 ] |
|
Author: {'name': 'Divjot Arora', 'email': 'divjot.arora@10gen.com', 'username': 'divjotarora'}Message: |
| Comment by Divjot Arora (Inactive) [ 28/Apr/20 ] |
| Comment by Divjot Arora (Inactive) [ 23/Apr/20 ] |
| Comment by Divjot Arora (Inactive) [ 22/Apr/20 ] |
|
Update: There is a ClusterClock passed down to the topology layer. It's used for heartbeats, but not handshakes. The current solution is to set the ClusterClock for non-auth handshakers and add a field to auth.HandshakeOptions for auth handshakers. |