Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bench: BenchmarkAlterTableDropColumn fails: got 21.142857142857142, expected 22 #70896

Closed
andreimatei opened this issue Sep 29, 2021 · 3 comments
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. C-test-failure Broken test (automatically or manually discovered). T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@andreimatei
Copy link
Contributor

Do make bench PKG=./pkg/bench/rttanalysis TESTFLAGS="-v" BENCHES=BenchmarkAlterTableDropColumn and see that multiple benchmarks fail. They seem to fail in the good direction - less roundtrips.
I think other benchmarks in the rttanalysis pkg are affected too.

I think this bit me in CI; CI runs the benchmarks with -benchtime=1ms. That seems to usually work, but I guess you can get unlucky.

@andreimatei andreimatei added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. C-test-failure Broken test (automatically or manually discovered). labels Sep 29, 2021
@ajwerner
Copy link
Contributor

I think this bit me in CI; CI runs the benchmarks with -benchtime=1ms. That seems to usually work, but I guess you can get unlucky.

This test is actually a bit meta, the go test runs itself in bench mode.

flags := []string{
"--test.run=^$",
"--test.bench=" + b,
"--test.benchtime=1x",
}

@andreimatei
Copy link
Contributor Author

TIL benchtime=1x

@postamar postamar removed their assignment Jan 26, 2022
@blathers-crl blathers-crl bot added the T-sql-schema-deprecated Use T-sql-foundations instead label Jan 26, 2022
@ajwerner
Copy link
Contributor

ajwerner commented Feb 1, 2022

Stale.

@ajwerner ajwerner closed this as completed Feb 1, 2022
@exalate-issue-sync exalate-issue-sync bot added T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) and removed T-sql-schema-deprecated Use T-sql-foundations instead labels May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. C-test-failure Broken test (automatically or manually discovered). T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

No branches or pull requests

3 participants