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

sql: add ALTER TYPE DROP VALUE to SQLSmith #62812

Closed
arulajmani opened this issue Mar 30, 2021 · 0 comments · Fixed by #62883
Closed

sql: add ALTER TYPE DROP VALUE to SQLSmith #62812

arulajmani opened this issue Mar 30, 2021 · 0 comments · Fixed by #62883
Assignees
Labels
C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior.

Comments

@arulajmani
Copy link
Collaborator

Describe the problem

We introduced ALTER TYPE ... DROP VALUE in 21.1 and to that effect we should add support for this statement to SQLSmith.

@arulajmani arulajmani added the C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. label Mar 30, 2021
@arulajmani arulajmani self-assigned this Mar 30, 2021
craig bot pushed a commit that referenced this issue Apr 1, 2021
61948: roachtest/cdc: update status when watching changefeed latency r=andreimatei a=andreimatei

When watching the latency, we were logging every second. This patch
reduces the frequency to 1/10s and makes the test also update its status
so that the current latency high-water mark is visible in the web UI.

Release note: None

62166: sql: add roach test / workload for testing connection latencies r=rafiss a=RichardJCai

Add roach test and workload test for testing connection latencies.

Release note: None

Making it a private test right now (not available through `cockroach workload`)

Need to also add it to teamcity and make sure the results are reported in roachperf, will open PR there.

Resolves #59394


62287: cdc: Improve avro encoder performance.  r=miretskiy a=miretskiy

Add avro encoder benchmarks.

Avoid expensive allocations (maps) when encoding datums.
Improve encoder performance by ~40%, and significantly reduce
memory allocations per op.

Release Notes: None

62883: sqlsmith: add support for ALTER TYPE ... DROP VALUE r=rafiss a=arulajmani

Now that `ALTER TYPE ... DROP VALUE` is a thing, let's add it to SQL
Smith.

Closes #62812

Release note: None

62922: roachtest/tpccbench: don't require workload binary r=andreimatei a=andreimatei

tpccbench was copying around the `workload` binary, but it then doesn't
use it; it uses `cockroach workload ...`. This patch removes the
copying.

Release note: None

62925: colexec: optimize CASE, AND, and OR projection operators r=yuzefovich a=yuzefovich

Previously, in CASE, AND, and OR projection operators we would always
allocate internal selection vectors of maximum capacity in the
constructor. This is inefficient when the query processes only small
data sets. This commit makes the allocations lazy and of tight sizes
based on the batch lengths.

Addresses: #62212

Release note: None

62926: bench,sql: add logScope to some benchmarks r=yuzefovich a=yuzefovich

This commits adds `defer log.Scope(b).Close(b)` to some benchmarks in
`bench`, `colexec`, and `rowexec` packages. Note that in some cases this
might not matter, but I didn't think through each carefully since it
shouldn't do any harm.

Release note: None

62962: sql: fix bug in close of ieSyncResultChannel r=yuzefovich a=ajwerner

When finishing sending on the ieSyncResultChannel we do not need to and should
not close the channel used to unblock the sender. Doing so can result in a
panic if the reader is concurrently trying to unblock the sender. We could
close that channel but there's no obvious reason to.

Fixes #62939

Release note: None

Co-authored-by: Andrei Matei <[email protected]>
Co-authored-by: richardjcai <[email protected]>
Co-authored-by: Yevgeniy Miretskiy <[email protected]>
Co-authored-by: arulajmani <[email protected]>
Co-authored-by: Yahor Yuzefovich <[email protected]>
Co-authored-by: Andrew Werner <[email protected]>
@craig craig bot closed this as completed in 79f0078 Apr 1, 2021
arulajmani added a commit to arulajmani/cockroach that referenced this issue Apr 1, 2021
Now that `ALTER TYPE ... DROP VALUE` is a thing, let's add it to SQL
Smith.

Closes cockroachdb#62812

Release note: None
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.

1 participant