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

NOT BETWEEN query on VIEW results in an internal error with VECTORIZE=experimental_on #44822

Closed
mrigger opened this issue Feb 6, 2020 · 0 comments · Fixed by #44829
Closed
Assignees
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@mrigger
Copy link

mrigger commented Feb 6, 2020

The following statements unexpectedly result in an internal error (see further below):

SET SESSION VECTORIZE=experimental_on;
CREATE TABLE t0(c0 BYTES);
CREATE VIEW v0(c0) AS SELECT MIN(t0.c0) FROM t0;
SELECT * FROM v0 WHERE v0.c0 NOT BETWEEN v0.c0 AND v0.c0; -- ERROR: internal error: unexpected error from the vectorized runtime: runtime error: index out of range [2047] with length 1024
ERROR: internal error: unexpected error from the vectorized runtime: runtime error: index out of range [2047] with length 1024
SQLSTATE: XX000
DETAIL: stack trace:
github.com/cockroachdb/cockroach/pkg/sql/colexec/execerror/error.go:70: func1()
runtime/panic.go:679: gopanic()
runtime/panic.go:75: goPanicIndex()
github.com/cockroachdb/cockroach/pkg/sql/colexec/proj_non_const_ops.eg.go:1086: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/bool_vec_to_sel.go:125: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/bool_vec_to_sel.go:36: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/const.eg.go:124: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/case.go:136: func1()
github.com/cockroachdb/cockroach/pkg/sql/colexec/allocator.go:116: PerformOperation()
github.com/cockroachdb/cockroach/pkg/sql/colexec/case.go:131: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/bool_vec_to_sel.go:125: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/bool_vec_to_sel.go:36: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/simple_project.go:93: Next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/materializer.go:149: next()
github.com/cockroachdb/cockroach/pkg/sql/colexec/materializer.go:140: nextAdapter()
github.com/cockroachdb/cockroach/pkg/sql/colexec/execerror/error.go:91: CatchVectorizedRuntimeError()
github.com/cockroachdb/cockroach/pkg/sql/colexec/materializer.go:177: Next()
github.com/cockroachdb/cockroach/pkg/sql/execinfra/base.go:170: Run()
github.com/cockroachdb/cockroach/pkg/sql/execinfra/processorsbase.go:749: Run()
github.com/cockroachdb/cockroach/pkg/sql/flowinfra/flow.go:370: Run()
github.com/cockroachdb/cockroach/pkg/sql/distsql_running.go:377: Run()
github.com/cockroachdb/cockroach/pkg/sql/distsql_running.go:987: PlanAndRun()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:834: execWithDistSQLEngine()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:724: dispatchToExecutionEngine()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:420: execStmtInOpenState()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor_exec.go:93: execStmt()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1316: execCmd()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1245: run()
github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:452: ServeConn()
github.com/cockroachdb/cockroach/pkg/sql/pgwire/conn.go:566: func1()
runtime/asm_amd64.s:1357: goexit()

I found this bug based on commit 4f37cfec944e584a2765819d9413a91c122fa201.

@yuzefovich yuzefovich added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Feb 6, 2020
@yuzefovich yuzefovich added this to Triage in BACKLOG, NO NEW ISSUES: SQL Execution via automation Feb 6, 2020
@yuzefovich yuzefovich self-assigned this Feb 6, 2020
@yuzefovich yuzefovich moved this from Triage to 20.1.6 in BACKLOG, NO NEW ISSUES: SQL Execution Feb 6, 2020
@craig craig bot closed this as completed in 82a0043 Feb 10, 2020
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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants