[release-12.3.1] Azure: Fix percentile syntax (#114707)
Azure: Fix `percentile` syntax (#114665)
Fix percentile syntax
(cherry picked from commit 1b10581c63)
Co-authored-by: Andreas Christou <andreas.christou@grafana.com>
This commit is contained in:
co-authored by
Andreas Christou
parent
855f376271
commit
c4604d6eb6
+1
-1
@@ -114,7 +114,7 @@ describe('AzureMonitorKustoQueryParser', () => {
|
||||
} as BuilderQueryExpression;
|
||||
|
||||
const result = AzureMonitorKustoQueryBuilder.toQuery(builderQuery);
|
||||
expect(result).toContain('summarize percentile(95, Duration)');
|
||||
expect(result).toContain('summarize percentile(Duration,95)');
|
||||
});
|
||||
|
||||
it('handles summarize with basic aggregation function like avg', () => {
|
||||
|
||||
+1
-1
@@ -86,7 +86,7 @@ const appendSummarize = (builderQuery: BuilderQueryExpression, phrases: string[]
|
||||
if (func === 'percentile') {
|
||||
const percentileValue = expr.parameters?.[0]?.value;
|
||||
const column = expr.parameters?.[1]?.value ?? expr.property?.name ?? '';
|
||||
return column ? `percentile(${percentileValue}, ${column})` : null;
|
||||
return column ? `percentile(${column},${percentileValue})` : null;
|
||||
}
|
||||
|
||||
const column = expr.property?.name ?? '';
|
||||
|
||||
Reference in New Issue
Block a user