Skip to content

fix: error when using materialized CTE in multi-statement transactions - #18707

Merged
bohutang merged 2 commits into
databendlabs:mainfrom
SkyFan2002:add_log
Sep 10, 2025
Merged

bohutang merged 2 commits into
databendlabs:mainfrom
SkyFan2002:add_log

Conversation

@SkyFan2002

@SkyFan2002 SkyFan2002 commented Sep 10, 2025 •

Copy link
Copy Markdown
Member

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

Summary

CTE temp table should be properly cleaned up in txnbuffer.

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 Sep 10, 2025
@github-actions

github-actions Bot commented Sep 10, 2025 •

Copy link
Copy Markdown
Contributor

🤖 Smart Auto-retry Analysis

Workflow: 17601040708

📊 Summary

  • Total Jobs: 83
  • Failed Jobs: 1
  • Retryable: 0
  • Code Issues: 1

❌ NO RETRY NEEDED

All failures appear to be code/test issues requiring manual fixes.

🔍 Job Details

  • ❌ linux / test_private_tasks: Not retryable (Code/Test)

🤖 About

Automated analysis using job annotations to distinguish infrastructure issues (auto-retried) from code/test issues (manual fixes needed).

@bohutang
bohutang merged commit dfad4bc into databendlabs:main Sep 10, 2025
132 of 139 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix this PR patches a bug in codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants