Skip to content

optimizer: Downgrade repr type error: sql_union to trace#35028

Merged
ggevay merged 1 commit intoMaterializeInc:mainfrom
ggevay:donwgrade-repr-type-error
Feb 16, 2026
Merged

optimizer: Downgrade repr type error: sql_union to trace#35028
ggevay merged 1 commit intoMaterializeInc:mainfrom
ggevay:donwgrade-repr-type-error

Conversation

@ggevay
Copy link
Contributor

@ggevay ggevay commented Feb 16, 2026 •
edited
Loading

This turned out to be a super noisy log msg (happening many times at every system startup, both with cast elimination on and off), so downgrading it from error to trace, which is not shown by default. Discussed here: https://materializeinc.slack.com/archives/C08A62E0751/p1771245357416329?thread_ts=1771245144.471869&cid=C08A62E0751

@ggevay ggevay requested a review from a team as a code owner February 16, 2026 13:27
@github-actions
Copy link

github-actions bot commented Feb 16, 2026

Pre-merge checklist

  • The PR title is descriptive and will make sense in the git log.
  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).

@ggevay ggevay enabled auto-merge (squash) February 16, 2026 13:30
@ggevay ggevay merged commit a895c93 into MaterializeInc:main Feb 16, 2026
134 checks passed
@def- def- added the release-blocker Critical issue that should block *any* release if not fixed label Feb 16, 2026
@def- def- mentioned this pull request Feb 17, 2026
Merged
def- added a commit that referenced this pull request Feb 17, 2026
@def-
optimizer: Downgrade repr type error: base_eq failed for to trace (#…
0a202b6 
…35050)

Same as in #35028

Seen in
https://materializeinc.sentry.io/issues/7271635537/?query=is%3Aunresolved%20is%3Afor_review%20assigned_or_suggested%3A%5Bme%2C%20my_teams%2C%20none%5D&referrer=issue-stream&sort=inbox
bkirwi pushed a commit that referenced this pull request Feb 18, 2026
@ggevay @bkirwi
optimizer: Downgrade repr type error: sql_union to trace (#35028)
19de587 
This turned out to be a super noisy log msg (happening many times at
every system startup, both with cast elimination on and off), so
downgrading it from `error` to `trace`, which is not shown by default.
Discussed here:
https://materializeinc.slack.com/archives/C08A62E0751/p1771245357416329?thread_ts=1771245144.471869&cid=C08A62E0751
bkirwi pushed a commit that referenced this pull request Feb 18, 2026
@def- @bkirwi
optimizer: Downgrade repr type error: base_eq failed for to trace (#…
4a118ca 
…35050)

Same as in #35028

Seen in
https://materializeinc.sentry.io/issues/7271635537/?query=is%3Aunresolved%20is%3Afor_review%20assigned_or_suggested%3A%5Bme%2C%20my_teams%2C%20none%5D&referrer=issue-stream&sort=inbox
patrickwwbutler pushed a commit to patrickwwbutler/materialize that referenced this pull request Feb 19, 2026
@ggevay @patrickwwbutler
optimizer: Downgrade repr type error: sql_union to trace (Materiali…
ff634fb 
…zeInc#35028)

This turned out to be a super noisy log msg (happening many times at
every system startup, both with cast elimination on and off), so
downgrading it from `error` to `trace`, which is not shown by default.
Discussed here:
https://materializeinc.slack.com/archives/C08A62E0751/p1771245357416329?thread_ts=1771245144.471869&cid=C08A62E0751
patrickwwbutler pushed a commit to patrickwwbutler/materialize that referenced this pull request Feb 19, 2026
@def- @patrickwwbutler
optimizer: Downgrade repr type error: base_eq failed for to trace (M…
a0c7b9e 
…aterializeInc#35050)

Same as in MaterializeInc#35028

Seen in
https://materializeinc.sentry.io/issues/7271635537/?query=is%3Aunresolved%20is%3Afor_review%20assigned_or_suggested%3A%5Bme%2C%20my_teams%2C%20none%5D&referrer=issue-stream&sort=inbox
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@def- def- def- approved these changes

Assignees

No one assigned

Labels

release-blocker Critical issue that should block *any* release if not fixed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ggevay @def-