IntenseWebs/grafana
3
0
Fork 0
mirror of https://github.com/grafana/grafana.git synced 2025-02-14 01:23:32 -06:00
Code Issues Packages Projects Releases Wiki Activity

Tempo: Replace template variables in TraceQL tab when streaming is enabled (#73259)

Browse Source
This commit is contained in:
Fabrizio 2023-08-24 14:10:14 +02:00 committed by GitHub
parent ed8ca02e0f
commit d293b08e52
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
public/app/plugins/datasource/tempo/datasource.ts
View File

@ -228,7 +228,7 @@ export class TempoDatasource extends DataSourceWithBackend<TempoQuery, TempoJson
});
if (config.featureToggles.traceQLStreaming) {
subQueries.push(this.handleStreamingSearch(options, targets.traceql));
subQueries.push(this.handleStreamingSearch(options, targets.traceql, queryValue));
} else {
subQueries.push(
this._request('/api/search', {
@ -430,22 +430,21 @@ export class TempoDatasource extends DataSourceWithBackend<TempoQuery, TempoJson
return request;
}
// This function can probably be simplified by avoiding passing both `targets` and `query`,
// since `query` is built from `targets`, if you look at how this function is currently called
handleStreamingSearch(
options: DataQueryRequest<TempoQuery>,
targets: TempoQuery[],
query?: string
query: string
): Observable<DataQueryResponse> {
const validTargets = targets
.filter((t) => t.query || query)
.map((t): TempoQuery => ({ ...t, query: query || t.query.trim() }));
if (!validTargets.length) {
if (query === '') {
return EMPTY;
}
return merge(
...validTargets.map((q) =>
...targets.map((target) =>
doTempoChannelStream(
q,
{ ...target, query },
this, // the datasource
options,
this.instanceSettings