release-26.1: rowenc: fix panic in composite datum type checking#161747
release-26.1: rowenc: fix panic in composite datum type checking#161747yuzefovich merged 1 commit intorelease-26.1from
Conversation
Fixes #161402 Previously, composite type checking assumed all values implemented tree.CompositeDatum, causing panics when this wasn't true. Use safe type assertion to check if datum is composite before calling IsComposite(). Release note: None
bf5157c to
6e9b787
Compare
|
Thanks for opening a backport. Before merging, please confirm that the change does not break backwards compatibility and otherwise complies with the backport policy. Include a brief release justification in the PR description explaining why the backport is appropriate. All backports must be reviewed by the TL for the owning area. While the stricter LTS policy does not yet apply, please exercise judgment and consider gating non-critical changes behind a disabled-by-default feature flag when appropriate. |
|
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
michae2
left a comment
There was a problem hiding this comment.
@michae2 reviewed 1 file and all commit messages, and made 1 comment.
Reviewable status:complete! 1 of 0 LGTMs obtained (waiting on @mw5h and @yuzefovich).
|
Reminder: it has been 2 weeks please merge or close your backport! |
Backport 1/1 commits from #161739 on behalf of @ZhouXing19.
Fixes #161402
Previously, composite type checking assumed all values implemented tree.CompositeDatum, causing panics when this wasn't true. Use safe type assertion to check if datum is composite before calling IsComposite().
Release note: None
Release justification: fix for high severity stability issue that causes node crashes. A test failure revealed that incorrect type assumptions in rowenc can trigger panics when processing composite types, leading to node instability and potential service disruption.