Skip to content

fix: rename PanicError and fix executor OOM mapping#19614

Merged
zhang2014 merged 2 commits intodatabendlabs:mainfrom
sundy-li:issue-19612-20260325-0943
Mar 27, 2026
Merged

fix: rename PanicError and fix executor OOM mapping#19614
zhang2014 merged 2 commits intodatabendlabs:mainfrom
sundy-li:issue-19612-20260325-0943

Conversation

@sundy-li
Copy link
Copy Markdown
Member

@sundy-li sundy-li commented Mar 25, 2026

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Fixes #19612.

Rename ErrorCode::PanicError to ErrorCode::UnwindError while keeping error code 1104, and stop using that code for memory-limit flush failures in the pipeline executor.

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

@github-actions github-actions bot added the pr-bugfix this PR patches a bug in codebase label Mar 25, 2026
@sundy-li sundy-li added the agent-reviewable Ready for agent review label Mar 25, 2026
@sundy-li sundy-li added agent-approved Approved by agent agent-changed Changed by agent and removed agent-reviewable Ready for agent review agent-approved Approved by agent labels Mar 25, 2026
@sundy-li sundy-li added agent-reviewable Ready for agent review agent-approved Approved by agent and removed agent-changed Changed by agent agent-reviewable Ready for agent review labels Mar 26, 2026
@zhang2014 zhang2014 merged commit 4a53f73 into databendlabs:main Mar 27, 2026
106 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-approved Approved by agent pr-bugfix this PR patches a bug in codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename ErrorCode::PanicError to reduce confusion and fix misuse

2 participants