[CDRIVER-4085] Add server connectionId to command monitoring events Created: 15/Jul/21 Updated: 28/Oct/23 Resolved: 28/Apr/22 |
|
| Status: | Closed |
| Project: | C Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.22.0, 1.22.0-beta0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Backlog - Core Eng Program Management Team | Assignee: | Roberto Sanchez |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Quarter: | FY22Q3 | ||||||||||||||||||||||||||||
| Upstream Changes Summary: | DRIVERS-1141: |
||||||||||||||||||||||||||||
| Description |
|
In scope of
The connectionid reported in the hello response will be 2. Drivers should sync their command monitoring tests with this commit, update their APM logic to append a new serverConnectionId field to commandStartedEvent, commandSucceededEvent and commandFailedEvent when it is available, and update their test runners to handle UTF schema version 1.6 (featuring the new hasServerConnectionId assertion).
|
| Comments |
| Comment by Githook User [ 28/Apr/22 ] |
|
Author: {'name': 'Jeremy Mikola', 'email': 'jmikola@gmail.com', 'username': 'jmikola'}Message: |
| Comment by Jeremy Mikola [ 27/Apr/22 ] |
| Comment by Githook User [ 26/Apr/22 ] |
|
Author: {'name': 'Roberto C. Sánchez', 'email': 'roberto@connexer.com', 'username': 'rcsanchez97'}Message:
Co-authored-by: Ezra Chung <88335979+eramongodb@users.noreply.github.com>
Co-authored-by: Ezra Chung <88335979+eramongodb@users.noreply.github.com> |
| Comment by Jeremy Mikola [ 31/Mar/22 ] |
|
kevin.albertson: I propose we schedule this sooner than later, as it is tied to unified test runner schema version 1.6, which may be a prerequisite for the change stream spec work relying upon version 1.7 (DRIVERS-1713). I will revert this to "Needs Triage" for consideration. |