Skip to content

Revert "admission: add per-group CPU time token metrics"#171136

Draft
wenyihu6 wants to merge 2 commits into
cockroachdb:masterfrom
wenyihu6:revertperf
Draft

Revert "admission: add per-group CPU time token metrics"#171136
wenyihu6 wants to merge 2 commits into
cockroachdb:masterfrom
wenyihu6:revertperf

Conversation

@wenyihu6

Copy link
Copy Markdown
Contributor

This reverts commit 37a57db.

@trunk-io

trunk-io Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Merging to master in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@cockroach-teamcity

Copy link
Copy Markdown
Member

This change is Reviewable

@wenyihu6 wenyihu6 added the X-perf-check Microbenchmarks CI: Added to a PR if a performance regression is detected and should be checked label May 28, 2026
@cockroach-teamcity

Copy link
Copy Markdown
Member
⚪ Sysbench [SQL, 3node, oltp_read_write]
Metric Old Commit New Commit Delta Note
sec/op 9.304m ±1% 9.310m ±1% ~ p=0.967 n=15
allocs/op 6.293k ±0% 6.293k ±0% ~ p=0.751 n=15
Reproduce

benchdiff binaries:

mkdir -p benchdiff/34b011a/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/34b011a2522b2081abe5741426b52b6488a8599b/bin/pkg_sql_tests benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/f7aa42b/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/f7aa42b56865f3a04a2512fdbff1d89e090c77e0/bin/pkg_sql_tests benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests

benchdiff command:

# NB: for best (most stable) results, also add a suitable `--benchtime` that
# results in ~1s to ~5s of benchmark runs. For example, if ops average ~3ms, a
# benchtime of `1000x` is appropriate.
#
# Some benchmarks (in particular BenchmarkSysbench) output additional memory
# profiles covering only the execution (excluding the setup/teardown) - those
# should be preferred for analysis since they more closely correspond to what's
# reported as B/op and alloc/op.
benchdiff --run=^BenchmarkSysbench/SQL/3node/oltp_read_write$ --old=f7aa42b --new=34b011a --memprofile ./pkg/sql/tests
⚪ Sysbench [KV, 3node, oltp_read_only]
Metric Old Commit New Commit Delta Note
sec/op 3.104m ±1% 3.112m ±1% ~ p=0.217 n=15
allocs/op 2.102k ±0% 2.102k ±0% ~ p=0.576 n=15
Reproduce

benchdiff binaries:

mkdir -p benchdiff/34b011a/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/34b011a2522b2081abe5741426b52b6488a8599b/bin/pkg_sql_tests benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/f7aa42b/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/f7aa42b56865f3a04a2512fdbff1d89e090c77e0/bin/pkg_sql_tests benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests

benchdiff command:

# NB: for best (most stable) results, also add a suitable `--benchtime` that
# results in ~1s to ~5s of benchmark runs. For example, if ops average ~3ms, a
# benchtime of `1000x` is appropriate.
#
# Some benchmarks (in particular BenchmarkSysbench) output additional memory
# profiles covering only the execution (excluding the setup/teardown) - those
# should be preferred for analysis since they more closely correspond to what's
# reported as B/op and alloc/op.
benchdiff --run=^BenchmarkSysbench/KV/3node/oltp_read_only$ --old=f7aa42b --new=34b011a --memprofile ./pkg/sql/tests
⚪ Sysbench [KV, 3node, oltp_write_only]
Metric Old Commit New Commit Delta Note
sec/op 2.852m ±1% 2.853m ±1% ~ p=0.775 n=15
allocs/op 4.204k ±0% 4.206k ±0% ~ p=0.116 n=15
Reproduce

benchdiff binaries:

mkdir -p benchdiff/34b011a/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/34b011a2522b2081abe5741426b52b6488a8599b/bin/pkg_sql_tests benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/34b011a/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
mkdir -p benchdiff/f7aa42b/bin/1058449141
gcloud storage cp gs://cockroach-microbench-ci/builds/f7aa42b56865f3a04a2512fdbff1d89e090c77e0/bin/pkg_sql_tests benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests
chmod +x benchdiff/f7aa42b/bin/1058449141/cockroachdb_cockroach_pkg_sql_tests

benchdiff command:

# NB: for best (most stable) results, also add a suitable `--benchtime` that
# results in ~1s to ~5s of benchmark runs. For example, if ops average ~3ms, a
# benchtime of `1000x` is appropriate.
#
# Some benchmarks (in particular BenchmarkSysbench) output additional memory
# profiles covering only the execution (excluding the setup/teardown) - those
# should be preferred for analysis since they more closely correspond to what's
# reported as B/op and alloc/op.
benchdiff --run=^BenchmarkSysbench/KV/3node/oltp_write_only$ --old=f7aa42b --new=34b011a --memprofile ./pkg/sql/tests
Artifacts

download:

mkdir -p new
gcloud storage cp gs://cockroach-microbench-ci/artifacts/34b011a2522b2081abe5741426b52b6488a8599b/26594084651-1/\* new/
mkdir -p old
gcloud storage cp gs://cockroach-microbench-ci/artifacts/f7aa42b56865f3a04a2512fdbff1d89e090c77e0/26594084651-1/\* old/

built with commit: 34b011a2522b2081abe5741426b52b6488a8599b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

X-perf-check Microbenchmarks CI: Added to a PR if a performance regression is detected and should be checked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants