[CSHARP-2490] Reimplement SDAM to follow the spec more closely Created: 25/Jan/19 Updated: 31/Mar/22 |
|
| Status: | Backlog |
| Project: | C# Driver |
| Component/s: | Connectivity |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Epic | Priority: | Minor - P4 |
| Reporter: | Robert Stam | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | rp-track | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Quarter: | FY21Q4 | ||||||||||||||||||||||||||||||||
| Case: | (copied to CRM) | ||||||||||||||||||||||||||||||||
| Description |
|
The current implementation of heartbeats uses async code. It was also written before the SDAM spec was written. The code would be simpler if it used a dedicated thread and sync calls. A dedicated thread would also protect the SDAM code from being starved from running when an application misuses Tasks in a way that interferes with the SDAM Tasks from getting a chance to run. The SDAM code should be completely re-implemented to: 1. Follow the SDAM spec more closely which would involve using the same names and logic flow |
| Comments |
| Comment by Rachelle Palmer [ 11/Jun/20 ] |
|
HI alexvy86 - no problem, I have deleted the screenshots. |
| Comment by Alejandro Villarreal [ 13/May/20 ] |
|
Hmm I did not intend to upload those screenshots here, I probably dragged them to the wrong window by mistake. |
| Comment by Robert Stam [ 25/Jan/19 ] |
|
See the test program for |