[DRIVERS-2452] Add databaseName property to command events Created: 27/Sep/22 Updated: 03/Oct/23 |
|
| Status: | Implementing |
| Project: | Drivers |
| Component/s: | Command Logging and Monitoring |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Tom Selander | Assignee: | Qingyang Hu |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Changes: | Needed | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Downstream Changes Summary: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Start date: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
Original Go ticket report: Original DRIVERS ticket that was closed (for context): It was decided during triage of |
| Comments |
| Comment by Githook User [ 26/Jun/23 ] |
|
Author: {'name': 'Qingyang Hu', 'email': '103950869+qingyang-hu@users.noreply.github.com', 'username': 'qingyang-hu'}Message: DRIVERS-2452 Add database name to the "command failed" and "command succeeded" log messages (#1436) |
| Comment by Githook User [ 21/Jun/23 ] |
|
Author: {'name': 'Qingyang Hu', 'email': '103950869+qingyang-hu@users.noreply.github.com', 'username': 'qingyang-hu'}Message: DRIVERS-2452 Add databaseName field to CommandFailedEvent and CommandSucceededEvent (#1434) |
| Comment by Jeffrey Yemin [ 08/Feb/23 ] |
|
What about CommandSucceededEvent? |
| Comment by Kaitlin Mahar [ 27/Sep/22 ] |
|
Note that to test this we probably need to update the unified test format to add databaseName expectations to expectedCommandEvent.commandSucceededEvent and expectedCommandEvent.commandFailedEvent. |