Skip to content

feat(ddm): Add samples fallback query#64143

Merged
ArthurKnaus merged 1 commit intomasterfrom
aknaus/feat/ddm/samples-fallback-query
Jan 30, 2024
Merged

feat(ddm): Add samples fallback query#64143
ArthurKnaus merged 1 commit intomasterfrom
aknaus/feat/ddm/samples-fallback-query

Conversation

@ArthurKnaus
Copy link
Copy Markdown
Member

Add fallback request for fetching samples.
Pass query parameter to samples request in chart.
Log timeouts with ddm.correlated_samples.timeout counter.

The timeout on the FE is 15 seconds after that we will show samples of the last hour of the selected timeframe.
The timeout mechanism is only enabled if the selected timeframe is bigger than 1 hour.

This is supposed to be a temporary measure until the endpoint is faster

@ArthurKnaus ArthurKnaus requested a review from a team as a code owner January 30, 2024 08:44
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 30, 2024
@ArthurKnaus ArthurKnaus enabled auto-merge (squash) January 30, 2024 08:53
@ArthurKnaus ArthurKnaus merged commit 9df6d87 into master Jan 30, 2024
@ArthurKnaus ArthurKnaus deleted the aknaus/feat/ddm/samples-fallback-query branch January 30, 2024 08:56
snigdhas pushed a commit that referenced this pull request Jan 30, 2024
Add fallback request for fetching samples.
Pass query parameter to samples request in chart.
Log timeouts with `ddm.correlated_samples.timeout` counter.

The timeout on the FE is 15 seconds after that we will show samples of
the last hour of the selected timeframe.
The timeout mechanism is only enabled if the selected timeframe is
bigger than 1 hour.

- closes #64057
@github-actions github-actions bot locked and limited conversation to collaborators Feb 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shorter fallback query for samples

2 participants