Skip to content

Commit

Permalink
Merge #61719
Browse files Browse the repository at this point in the history
61719: ui: add contention graph to SQL metrics overview page r=yuzefovich,awoods187 a=asubiotto

Release justification: low-risk, high-benefit change to existing functionality.
Exposes a very useful graph that was hidden away.

Release note (sql change): a statement contention timeseries is now displayed
in the SQL metrics overview dashboard.

Closes #60156

Screenshot:
![image](https://user-images.githubusercontent.com/10560359/110498965-cb245280-80c5-11eb-9f2e-276b6575e2bb.png)


Co-authored-by: Alfonso Subiotto Marques <[email protected]>
  • Loading branch information
craig[bot] and asubiotto committed Mar 11, 2021
2 parents b3812b5 + 2442bc2 commit be50141
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions pkg/ui/src/views/cluster/containers/nodeGraphs/dashboards/sql.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,20 @@ export default function (props: GraphDashboardProps) {
</Axis>
</LineGraph>,

<LineGraph
title="SQL Statement Contention"
sources={nodeSources}
tooltip={`The total number of SQL statements that experienced contention ${tooltipSelection}.`}
>
<Axis label="queries">
<Metric
name="cr.node.sql.distsql.contended_queries.count"
title="Contention"
nonNegativeRate
/>
</Axis>
</LineGraph>,

<LineGraph
title="Active Flows for Distributed SQL Statements"
tooltip="The number of flows on each node contributing to currently running distributed SQL statements."
Expand Down

0 comments on commit be50141

Please sign in to comment.