Skip to content

feat: stmt support branch#19774

Draft
zhyass wants to merge 21 commits intodatabendlabs:mainfrom
zhyass:feat_branch
Draft

feat: stmt support branch#19774
zhyass wants to merge 21 commits intodatabendlabs:mainfrom
zhyass:feat_branch

Conversation

@zhyass
Copy link
Copy Markdown
Member

@zhyass zhyass commented Apr 26, 2026

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

Summary

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

@zhyass zhyass requested a review from drmingdrmer as a code owner April 26, 2026 15:47
@zhyass zhyass marked this pull request as draft April 26, 2026 15:47
@github-actions github-actions Bot added the pr-feature this PR introduces a new feature to the codebase label Apr 26, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4c0e0a2811

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

continue;
}
match catalog
.get_table_branch(&tenant, &db.name, base_table, branch_name, false)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Enforce table-ref gate before resolving branch filters

This branch-resolution path calls catalog.get_table_branch(...) directly, which bypasses the normal QueryContext lookup path that applies check_table_ref_access (session flag + enterprise gate). In practice, users can still read branch column metadata via system.columns/SHOW COLUMNS branch filters even when enable_experimental_table_ref=0, so the feature gate is no longer consistently enforced.

Useful? React with 👍 / 👎.

branch support recluster falshback constraint row access

branch support compact and analyze

index support branch

fix

branch support dml write

truncate refresh virtual column and insert multi table

create index

fix

support undrop branch

add system.branches and show branches

fix
@github-actions
Copy link
Copy Markdown
Contributor

🤖 CI Job Analysis

Workflow: 25180431892

📊 Summary

  • Total Jobs: 26
  • Failed Jobs: 3
  • Retryable: 0
  • Code Issues: 3

NO RETRY NEEDED

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

🔍 Job Details

  • mac_check: Not retryable (Code/Test)
  • linux / test_unit: Not retryable (Code/Test)
  • linux / check: Not retryable (Code/Test)

🤖 About

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature this PR introduces a new feature to the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant