-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Fix object name field in rpc query_completion events and activity_id fields for error events #20302
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
/merge |
View all feedbacks in Devflow UI.
This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
devflow unqueued this merge request: It did not become mergeable within the expected time |
Review from lu-zhengda is dismissed. Related teams and files:
- database-monitoring-agent
- sqlserver/changelog.d/20302.fixed
- sqlserver/datadog_checks/sqlserver/xe_collection/base.py
- sqlserver/datadog_checks/sqlserver/xe_collection/error_events.py
- sqlserver/datadog_checks/sqlserver/xe_collection/query_completion_events.py
- sqlserver/tests/test_xe_collection.py
- sqlserver/tests/xml_xe_events/attention.xml
- sqlserver/tests/xml_xe_events/error_reported.xml
…fields for error events (#20302) * include object_name, not procedure name for rpc events * add changelog * fix changelog * object id and spills also not real fields * Include duration and query start for attention events * activity id and activity id xfer * add activity_id_xfer to base string fields * changelogs * Update 20302.fixed (cherry picked from commit 1b01be5)
…fields for error events (DataDog#20302) * include object_name, not procedure name for rpc events * add changelog * fix changelog * object id and spills also not real fields * Include duration and query start for attention events * activity id and activity id xfer * add activity_id_xfer to base string fields * changelogs * Update 20302.fixed 1b01be5
…fields for error events (#20302) (#20317) * include object_name, not procedure name for rpc events * add changelog * fix changelog * object id and spills also not real fields * Include duration and query start for attention events * activity id and activity id xfer * add activity_id_xfer to base string fields * changelogs * Update 20302.fixed (cherry picked from commit 1b01be5) Co-authored-by: Allen Zhou <allen.zhou@datadoghq.com>
What does this PR do?
Fix object name field in rpc query_completion events. I was incorrectly using the field name of "procedure_name" when it is actually "object_name". Line number, object_id, and spills was also added from module_end but is not exposed in rpc events, so removed that field.
Also added activity_id_xfer field for all event types, which I discovered shows up in error events. Haven't seen it appear in query completion events, but adding it defensively.
Motivation
Review checklist (to be filled by reviewers)
qa/skip-qa
label if the PR doesn't need to be tested during QA.backport/<branch-name>
label to the PR and it will automatically open a backport PR once this one is merged