Skip to content

sql: Transparent Debug impl for StableRow to avoid noise in debug output - #38494

Merged
mgree merged 2 commits into
MaterializeInc:mainfrom
mgree:stablerow-simple-debug
Aug 26, 2026
Merged

sql: Transparent Debug impl for StableRow to avoid noise in debug output#38494
mgree merged 2 commits into
MaterializeInc:mainfrom
mgree:stablerow-simple-debug

Conversation

@mgree

@mgree mgree commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Motivation

Closes SQL-646.
#37814 (comment)

Description

The StableRow newtype struct derived Debug, which added the struct name as chatter. This PR removes it, per @antiguru's review.

Verification

Tests pass.

@mgree
mgree requested a review from a team as a code owner August 26, 2026 15:31
@mgree
mgree requested a review from antiguru August 26, 2026 15:31

@antiguru antiguru left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@mgree
mgree enabled auto-merge (squash) August 26, 2026 16:09
@mgree
mgree merged commit ec639cb into MaterializeInc:main Aug 26, 2026
78 checks passed
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.

2 participants