Skip to content

github: redirect KV code reviews to @cockroachdb/kv-prs#68903

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
irfansharif:210813.kv-codeowners
Aug 16, 2021
Merged

github: redirect KV code reviews to @cockroachdb/kv-prs#68903
craig[bot] merged 1 commit intocockroachdb:masterfrom
irfansharif:210813.kv-codeowners

Conversation

@irfansharif
Copy link
Copy Markdown
Contributor

We try to use the @cockroachdb/kv alias for notifying the entire team on
issues. There's no way to disable notifications for github's automatic
codeowners driven review requests, and that tends to be a firehose, so
lets use this sub-team alias instead.

Release note: None

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@irfansharif
Copy link
Copy Markdown
Contributor Author

Thanks!

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 13, 2021

Build failed (retrying...):

@rickystewart
Copy link
Copy Markdown
Collaborator

bors r-

Broken tests

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 13, 2021

Canceled.

We try to use the @cockroachdb/kv alias for notifying the entire team on
issues. There's no way to disable notifications for github's automatic
codeowners driven review requests, and that tends to be a firehose, so
lets use this sub-team alias instead.

Release note: None
@irfansharif irfansharif force-pushed the 210813.kv-codeowners branch from 13506d2 to d8d71d4 Compare August 13, 2021 23:43
@irfansharif
Copy link
Copy Markdown
Contributor Author

bors r+

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Aug 16, 2021

Build succeeded:

@craig craig bot merged commit f2aded5 into cockroachdb:master Aug 16, 2021
irfansharif added a commit to irfansharif/cockroach that referenced this pull request Aug 17, 2021
craig bot pushed a commit that referenced this pull request Aug 17, 2021
68678: opt: don't use outer columns as implicit grouping columns r=DrewKimball a=DrewKimball

Previously, the optbuilder logic would add any outer column that is
referenced in a grouping context to the set of grouping columns. This
is correct in some cases, because outer columns are effectively constant,
and can just be removed by norm rules. However, it is incorrect in the
case when there are no grouping columns, e.g. a `ScalarGroupBy`. In that
case, the `ScalarGroupBy` would inadvertently be converted into a `GroupBy`.
This patch modifies the optbuilder to simply ignore outer columns in a
grouping context.

Fixes #68290

Release note: None

68997: stats: use table descriptors instead of IDs r=postamar a=postamar

Previously, the sql stats package inferred various properties about
a table from its ID, like if it is a system or a virtual table.
However the table descriptor is usually readily or easily available,
providing much richer information about a table than its ID.
In particular, this allows us to stop collecting stats for views.

Release note (sql change): table statistics are no longer collected
for views.

69024: colexec: fix bytes corruption for disk-spilled window functions r=DrewKimball a=DrewKimball

This patch fixes the `Truncate` method for bytes columns so that it
updates the offsets to be non-decreasing up to the new `maxSetLength`.
This is necessary in the case when the new `maxSetLength` is greater
than the old one.

This can happen when a window function has a bytes output column and
the `SpillingQueue` that buffers input batches spills to disk. If a
batch has trailing nulls up to the last processed index, and it is
immediately enqueued to disk, `SpillingQueue` does not call `SetLength`
on the batch, so the offsets still need to be updated (in this case by
`Truncate`).

Fixes #60824

Release note: None

69039: github: route pkg/migration PRs to kv-prs sub-team r=irfansharif a=irfansharif

Missed one in #68903.

Release note: None

Co-authored-by: Drew Kimball <drewk@cockroachlabs.com>
Co-authored-by: Marius Posta <marius@cockroachlabs.com>
Co-authored-by: irfan sharif <irfanmahmoudsharif@gmail.com>
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.

4 participants