Skip to content

Commit

Permalink
Merge pull request #165 from natefoo/tool-metrics-multi
Browse files Browse the repository at this point in the history
Support specifying multiple metrics in query tool-metrics
  • Loading branch information
bgruening authored Oct 30, 2024
2 parents 8ae491e + ae0a424 commit ac3fd3f
Showing 1 changed file with 10 additions and 1 deletion.
11 changes: 10 additions & 1 deletion parts/22-query.sh
Original file line number Diff line number Diff line change
Expand Up @@ -1392,6 +1392,8 @@ query_tool-metrics() { ##? <tool_id> <metric_id> [last=-1] [--like] [--ok] [--su
[ 150.223, 162.733) n=746
Multiple metrics can be specified by separating them with a comma.
The optional 'last' argument can be used to limit the number of most recent jobs that will be checked.
The default is all jobs. Note that this can return zero results if your most recent jobs have not
returned metric data for your selected metric.
Expand All @@ -1402,6 +1404,7 @@ query_tool-metrics() { ##? <tool_id> <metric_id> [last=-1] [--like] [--ok] [--su
EOF

limit_clause=
metric_clause=
summary='*'

tool_subquery="SELECT id FROM job WHERE tool_id = '$arg_tool_id'"
Expand All @@ -1418,13 +1421,19 @@ query_tool-metrics() { ##? <tool_id> <metric_id> [last=-1] [--like] [--ok] [--su
summary="$(summary_statistics metric_value 0)"
fi

if [[ "$arg_metric_id" =~ ',' ]]; then
metric_clause="metric_name IN ('$(echo "$arg_metric_id" | sed "s/,/','/g")')"
else
metric_clause="metric_name = '$arg_metric_id'"
fi

read -r -d '' QUERY <<-EOF
WITH runtime_data AS (
SELECT
metric_value
FROM job_metric_numeric
WHERE
metric_name = '$arg_metric_id'
$metric_clause
and
job_id in (
$tool_subquery
Expand Down

0 comments on commit ac3fd3f

Please sign in to comment.