Skip to content

Conversation

@aerfrei
Copy link
Contributor

@aerfrei aerfrei commented Nov 20, 2025

Previously, DB-level feeds would emit messages with plain table neames, even when full_table_name was specified. Now, we respect this option.

Epic: none
Fixes: #158145

Release note: None

Previously, DB-level feeds would emit messages with plain
table neames, even when full_table_name was specified. Now,
we respect this option.

Epic: none
Fixes: cockroachdb#158145

Release note: None
@aerfrei aerfrei requested a review from log-head November 20, 2025 16:55
@aerfrei aerfrei requested a review from a team as a code owner November 20, 2025 16:55
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@log-head log-head left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@aerfrei aerfrei requested a review from asg0451 November 20, 2025 18:24
@aerfrei
Copy link
Contributor Author

aerfrei commented Nov 20, 2025

TFTR!

bors r=log-head,asg0451

craig bot pushed a commit that referenced this pull request Nov 20, 2025
158055: execstats: Extract AdmissionWorkQueueStats from traces r=alyshanjahani-crl a=alyshanjahani-crl

This commit extracts the AdmissionWorkQueueStats that we are recording on traces and adds it to a new field AdmissionWaitTime on the QueryLevelStats object.

The total admission wait time is now displayed in explain analyze.

A follow up PR will further plumb this through to SQL Stats.

Part of: https://cockroachlabs.atlassian.net/browse/CRDB-55925

Release note (sql change): Admission control wait time is now displayed as part of EXPLAIN ANALYZE when non zero.

158146: changefeedccl: respect full_table_name for db-level feeds r=log-head,asg0451 a=aerfrei

Previously, DB-level feeds would emit messages with plain table neames, even when full_table_name was specified. Now, we respect this option.

Epic: none
Fixes: #158145

Release note: None

Co-authored-by: alyshanjahani-crl <alyshan@cockroachlabs.com>
Co-authored-by: Aerin Freilich <aerin.freilich@cockroachlabs.com>
@craig
Copy link
Contributor

craig bot commented Nov 20, 2025

Build failed (retrying...):

@craig
Copy link
Contributor

craig bot commented Nov 20, 2025

@craig craig bot merged commit 9346663 into cockroachdb:master Nov 20, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CDC: database level feeds do not respect full_table_name option

4 participants