Skip to content

Correctly associate async queries with the HTTP request for Datadog - #7996

Merged
labkey-jeckels merged 5 commits into
release26.7-SNAPSHOTfrom
26.7_fb_datadogSpans
Aug 31, 2026
Merged

Correctly associate async queries with the HTTP request for Datadog#7996
labkey-jeckels merged 5 commits into
release26.7-SNAPSHOTfrom
26.7_fb_datadogSpans

Conversation

@labkey-jeckels

@labkey-jeckels labkey-jeckels commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Rationale

Datadog tracks queries executed for an HTTP request. We don't have the proper wiring to make the async query and aggregates connect to their HTTP request.

Changes

  • Switch how the spans are tracked
  • Improve naming for the spans
  • Intentionally keep materializing queries separate from the event that invoked them
  • Track errors and cancellations separately

Tasks

  • Claude Code Review
  • Manual Testing - N/A
  • Test Automation - N/A

@labkey-jeckels
labkey-jeckels requested a review from a team August 28, 2026 22:20
@labkey-jeckels
labkey-jeckels changed the base branch from develop to release26.7-SNAPSHOT August 28, 2026 22:20
@labkey-jeckels labkey-jeckels self-assigned this Aug 28, 2026
@LabKey LabKey deleted a comment from github-actions Bot Aug 28, 2026
@labkey-nicka labkey-nicka added this to the 26.07 milestone Aug 31, 2026
# Conflicts:
#	api/src/org/labkey/api/data/MaterializedQueryHelper.java
@labkey-jeckels
labkey-jeckels merged commit 1e8357e into release26.7-SNAPSHOT Aug 31, 2026
8 of 13 checks passed
@labkey-jeckels
labkey-jeckels deleted the 26.7_fb_datadogSpans branch August 31, 2026 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants