Skip to content

Conversation

@dantengsky
Copy link
Member

@dantengsky dantengsky commented Sep 18, 2025

Return early if found table has no snapshot

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

Summary

Return early during execution of fuse_vacuum2 if found that target table has no snapshot

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

…n_num_snapshots_to_keep policy

Return early if found table has no snapshot
@github-actions github-actions bot added the pr-bugfix this PR patches a bug in codebase label Sep 18, 2025
@dantengsky dantengsky marked this pull request as ready for review September 18, 2025 12:35
…s tracking

- Replace verbose TableMeta output with concise table_name(id:table_id) format
- Add clear start/completion markers with === delimiters
- Improve result summary with specific counts of success/failed operations
- Add detailed progress information while preserving all debug data
- Failed table IDs are still logged separately for troubleshooting
@dantengsky dantengsky merged commit c756b96 into databendlabs:main Sep 18, 2025
87 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.

bug: fuse_vacuum2 panic while vauuming empty table with data_retention_num_snapshots_to_keep policy

3 participants