Skip to content

Conversation

@TCeason
Copy link
Collaborator

@TCeason TCeason commented Nov 14, 2025

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

Summary

delete replace masking/row access policy

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-chore this PR only has small changes that no need to record, like coding styles. label Nov 14, 2025
@TCeason TCeason marked this pull request as draft November 14, 2025 06:32
@TCeason TCeason assigned TCeason and unassigned TCeason Nov 14, 2025
@TCeason TCeason force-pushed the remove_replace branch 3 times, most recently from 13c8897 to 7d34681 Compare November 17, 2025 00:42
@TCeason TCeason marked this pull request as ready for review November 17, 2025 01:38
@TCeason
Copy link
Collaborator Author

TCeason commented Nov 17, 2025

Wait pr: #18978 to fix ci error.

Copy link
Member

@drmingdrmer drmingdrmer left a comment

Choose a reason for hiding this comment

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

@drmingdrmer reviewed 15 of 17 files at r1, 2 of 2 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @KKould)

@TCeason TCeason merged commit 5220ad9 into databendlabs:main Nov 17, 2025
88 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-chore this PR only has small changes that no need to record, like coding styles.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants