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: BenchmarkFuncExprTypeCheck fails on master #97400

Closed
yuzefovich opened this issue Feb 21, 2023 · 2 comments · Fixed by #97755
Closed

bench: BenchmarkFuncExprTypeCheck fails on master #97400

yuzefovich opened this issue Feb 21, 2023 · 2 comments · Fixed by #97755
Assignees
Labels
branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. GA-blocker T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@yuzefovich
Copy link
Member

yuzefovich commented Feb 21, 2023

On 4806560 I get the following:

yuzefovich@crlMBP-YN9GWF6WF7NTI1 cockroach % ./dev bench pkg/bench -f=BenchmarkFuncExprTypeCheck -v --stream-output
$ bazel test pkg/bench:all --test_arg -test.run=- --test_arg -test.bench=BenchmarkFuncExprTypeCheck --test_sharding_strategy=disabled --test_arg -test.v --crdb_test_off --test_output streamed
INFO: Invocation ID: b39f29a0-15bb-4ea5-932b-7423e43f80c6
WARNING: Streamed test output requested. All tests will be run locally, without sharding, one at a time
INFO: Analyzed 3 targets (1 packages loaded, 216 targets configured).
INFO: Found 2 targets and 1 test target...
I230221 17:41:05.799887 1 (gostd) rand.go:199  [T1] 1  random seed: 2662163393425416388
goos: darwin
goarch: arm64
BenchmarkFuncExprTypeCheck
    test_log_scope.go:161: test logs captured to: /Users/yuzefovich/go/src/github.com/cockroachdb/cockroach/tmp/_tmp/43bfbdbe7df5c7b3e2a2cb71efa9af4e/logBenchmarkFuncExprTypeCheck242956656
    test_log_scope.go:79: use -show-logs to present logs inline
    bench_test.go:1443: error executing 'CREATE FUNCTION abs(val INT) RETURNS INT CALLED ON NULL INPUT LANGUAGE SQL AS $$ SELECT val $$': pq: cannot perform schema change on function abs(val: int) -> int because it is required by the database system
    panic.go:522: -- test log scope end --
test logs left over in: /Users/yuzefovich/go/src/github.com/cockroachdb/cockroach/tmp/_tmp/43bfbdbe7df5c7b3e2a2cb71efa9af4e/logBenchmarkFuncExprTypeCheck242956656
--- FAIL: BenchmarkFuncExprTypeCheck
FAIL
I230221 17:41:06.168402 1 (gostd) testmain.go:136  [T1] 1  Test //pkg/bench:bench_test exited with error code 1
FAIL: //pkg/bench:bench_test (see /private/var/tmp/_bazel_yahor/e311b1c2454b7c301a6b73bddff450e7/execroot/com_github_cockroachdb_cockroach/bazel-out/darwin_arm64-fastbuild/testlogs/pkg/bench/bench_test/test.log)
INFO: Elapsed time: 24.825s, Critical Path: 24.33s
INFO: 35 processes: 1 internal, 34 darwin-sandbox.
INFO: Build completed, 1 test FAILED, 35 total actions
//pkg/bench:bench_test                                                   FAILED in 1.7s
  /private/var/tmp/_bazel_yahor/e311b1c2454b7c301a6b73bddff450e7/execroot/com_github_cockroachdb_cockroach/bazel-out/darwin_arm64-fastbuild/testlogs/pkg/bench/bench_test/test.log

INFO: Build completed, 1 test FAILED, 35 total actions
ERROR: exit status 3

I'll skip the failing stuff for now.

Jira issue: CRDB-24682

@yuzefovich yuzefovich added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Feb 21, 2023
@blathers-crl blathers-crl bot added the T-sql-schema-deprecated Use T-sql-foundations instead label Feb 21, 2023
yuzefovich added a commit to yuzefovich/cockroach that referenced this issue Feb 21, 2023
@blathers-crl
Copy link

blathers-crl bot commented Feb 21, 2023

Hi @chengxiong-ruan, please add branch-* labels to identify which branch(es) this release-blocker affects.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@chengxiong-ruan chengxiong-ruan added the branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 label Feb 21, 2023
@chengxiong-ruan
Copy link
Contributor

woof...this was introduced recently I think, I'll have a fix for this. Basically UDF need to shadow builtins...

craig bot pushed a commit that referenced this issue Feb 21, 2023
97402: bench: skip a failing case in BenchmarkFuncExprTypeCheck r=yuzefovich a=yuzefovich

Informs: #97400.

Release note: None

Co-authored-by: Yahor Yuzefovich <[email protected]>
craig bot pushed a commit that referenced this issue Mar 2, 2023
97755: sql: set name prefix before resolving a function r=chengxiong-ruan a=chengxiong-ruan

Fixes: #97400

Previously, in desclarative schema changer, we would run into a builtin function when we try to create a UDF with a same builtin function signature. This is because we didn't set the name prefixes properly. This commit fixes this by setting the name prefixes using the prefix re-resolution result.

Release note: None

Co-authored-by: Chengxiong Ruan <[email protected]>
@craig craig bot closed this as completed in 38173eb Mar 2, 2023
@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
branch-release-23.1 Used to mark GA and release blockers, technical advisories, and bugs for 23.1 C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. GA-blocker T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants