Skip to content

Conversation

@discord9
Copy link
Contributor

@discord9 discord9 commented Sep 17, 2025

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

What's changed and what's your intention?

gc worker metasrv trigger, which trigger gc worker on datanode depending on file_removal_rate and sst_num

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

@github-actions github-actions bot added size/M docs-not-required This change does not impact docs. labels Sep 17, 2025
@discord9 discord9 force-pushed the gc_metasrv_trigger branch 3 times, most recently from 9f76fb4 to 6bfa276 Compare September 18, 2025 11:04
@github-actions github-actions bot added size/L and removed size/M labels Sep 18, 2025
@discord9 discord9 changed the title feat: gc worker metasrv trigger feat: gc worker metasrv scheduler Sep 19, 2025
@discord9 discord9 force-pushed the gc_metasrv_trigger branch 4 times, most recently from 4ac8dda to 3fa76b6 Compare October 15, 2025 03:09
@discord9 discord9 force-pushed the gc_metasrv_trigger branch 3 times, most recently from 92dc253 to ed00d04 Compare October 22, 2025 09:01
@github-actions github-actions bot added size/XL and removed size/L labels Oct 22, 2025
Signed-off-by: discord9 <[email protected]>

false by default

Signed-off-by: discord9 <[email protected]>

test: config api

Signed-off-by: discord9 <[email protected]>

refactor: per code review

Signed-off-by: discord9 <[email protected]>

less info!

Signed-off-by: discord9 <[email protected]>

even less info!!

Signed-off-by: discord9 <[email protected]>

docs: gc regions instr

Signed-off-by: discord9 <[email protected]>

refactor: grp by region id

Signed-off-by: discord9 <[email protected]>

per code review

Signed-off-by: discord9 <[email protected]>

per review

Signed-off-by: discord9 <[email protected]>

error handling?

Signed-off-by: discord9 <[email protected]>

test: fix

Signed-off-by: discord9 <[email protected]>

todos

Signed-off-by: discord9 <[email protected]>

aft rebase fix

Signed-off-by: discord9 <[email protected]>

after refactor

Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>

feat: dn file removal rate

Signed-off-by: discord9 <[email protected]>

feat: trigger gc with stats(WIP)

Signed-off-by: discord9 <[email protected]>

chore

Signed-off-by: discord9 <[email protected]>

also move files ref manifest to store-api

Signed-off-by: discord9 <[email protected]>

feat: basic gc trigger impl

Signed-off-by: discord9 <[email protected]>

wip: handle file ref change

Signed-off-by: discord9 <[email protected]>

refactor: use region ids

Signed-off-by: discord9 <[email protected]>

fix: retry using related regions

Signed-off-by: discord9 <[email protected]>

chore: rm unused

Signed-off-by: discord9 <[email protected]>

fix: update file reference type in GC worker

Signed-off-by: discord9 <[email protected]>

feat: dn gc limiter

Signed-off-by: discord9 <[email protected]>

rename

Signed-off-by: discord9 <[email protected]>

feat: gc scheduler retry with outdated regions

Signed-off-by: discord9 <[email protected]>

feat: use real object store purger

Signed-off-by: discord9 <[email protected]>

wip: add to metasrv

Signed-off-by: discord9 <[email protected]>

feat: add to metasrv

Signed-off-by: discord9 <[email protected]>

feat: datanode gc worker handler

Signed-off-by: discord9 <[email protected]>

fix: no partition col fix

Signed-off-by: discord9 <[email protected]>

fix: RegionId json deser workaround

Signed-off-by: discord9 <[email protected]>

fix: find access layer

Signed-off-by: discord9 <[email protected]>

fix: on host dn

Signed-off-by: discord9 <[email protected]>

fix: stat dedup

Signed-off-by: discord9 <[email protected]>

refactor: rm load-based

Signed-off-by: discord9 <[email protected]>

chore: aft rebase fix

Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>
Signed-off-by: discord9 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-required This change does not impact docs. size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant