cockroachdb / cockroachdb/cockroach

Add metrics for sql service latency broken down by statement type

Open
#118,534 1 comment 0 reactions 0 assignees View on GitHub
C-enhancement E-quick-win T-sql-queries
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

Today we have SQL service latency metric (called `sql_service_latency_bucket` in prometheus) for overall SQL traffic. Some customers find it valuable to break this down by SELECT, INSERT, DELETE, UPDATE etc so they can understand performance characteristics better. To quote a customer:
>The idea is that update, insert, select, and to some extent delete have different > performance characteristics and resource needs, so it might be useful to have e.g. >cr.node.sql.insert_service.latency-p50, cr.node.sql.update_service.latency-p50, >cr.node.sql.select_service.latency-p50 instead of just cr.node.sql.service.latency-p50

Jira issue: CRDB-35777

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.