Skip to content

sqlsmith: tell sqlsmith about more things that vectorized supports#41075

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
yuzefovich:sqlsmith-vec
Sep 25, 2019
Merged

sqlsmith: tell sqlsmith about more things that vectorized supports#41075
craig[bot] merged 1 commit intocockroachdb:masterfrom
yuzefovich:sqlsmith-vec

Conversation

@yuzefovich
Copy link
Copy Markdown
Member

The vectorized engine currently has support for AndExpr, OrExpr,
CaseExpr, Like and NotLike, InExpr, and all of these things are now
added to sqlsmith when vectorizable is true.

Release justification: Category 1: Non-production code changes.

Release note: None

The vectorized engine currently has support for AndExpr, OrExpr,
CaseExpr, Like and NotLike, InExpr, and all of these things are now
added to sqlsmith when vectorizable is true.

Release justification: Category 1: Non-production code changes.

Release note: None
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@yuzefovich
Copy link
Copy Markdown
Member Author

I think some of these things we do not fully support (for example, something like this CASE WHEN true THEN NULL ELSE 1 END will not be run with vec because of "untyped null"), but maybe it's ok to merge this as is and see what sqlsmith finds?

@yuzefovich
Copy link
Copy Markdown
Member Author

Thanks for the review!

bors r=mjibson

craig bot pushed a commit that referenced this pull request Sep 25, 2019
41075: sqlsmith: tell sqlsmith about more things that vectorized supports r=mjibson a=yuzefovich

The vectorized engine currently has support for AndExpr, OrExpr,
CaseExpr, Like and NotLike, InExpr, and all of these things are now
added to sqlsmith when vectorizable is true.

Release justification: Category 1: Non-production code changes.

Release note: None

Co-authored-by: Yahor Yuzefovich <yahor@cockroachlabs.com>
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Sep 25, 2019

Build succeeded

@craig craig bot merged commit 69ef011 into cockroachdb:master Sep 25, 2019
@yuzefovich yuzefovich deleted the sqlsmith-vec branch November 12, 2019 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants