Page MenuHomePhabricator

Provide `bin/garbage` for interacting with garbage collection
ClosedPublic

Authored by epriestley on Oct 1 2015, 11:17 PM.
Tags
None
Referenced Files
F19472676: D14219.diff
Wed, Jan 7, 11:46 AM
F18884684: D14219.diff
Nov 7 2025, 1:38 AM
F18830095: D14219.diff
Oct 25 2025, 4:09 AM
F18817426: D14219.id.diff
Oct 21 2025, 3:29 PM
F18774246: D14219.id.diff
Oct 9 2025, 6:07 PM
F18771607: D14219.diff
Oct 8 2025, 8:38 PM
F18736555: D14219.id34332.diff
Oct 1 2025, 6:27 AM
F18638790: D14219.id34332.diff
Sep 17 2025, 11:31 AM
Subscribers
None

Details

Summary

Fixes T9494. This:

  • Removes all the random GC.x.y.z config.
  • Puts it all in one place that's locked and which you use bin/garbage set-policy ... to adjust.
  • Makes every TTL-based GC configurable.
  • Simplifies the code in the actual GCs.
Test Plan
  • Ran bin/garbage collect to collect some garbage, until it stopped collecting.
  • Ran bin/garbage set-policy ... to shorten policy. Saw change in web UI. Ran bin/garbage collect again and saw it collect more garbage.
  • Set policy to indefinite and saw it not collect garabge.
  • Set policy to default and saw it reflected in web UI / collect.
  • Ran bin/phd debug trigger and saw all GCs fire with reasonable looking queries.
  • Read new docs.

Screen Shot 2015-10-01 at 4.16.42 PM.png (1×1 px, 268 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Provide `bin/garbage` for interacting with garbage collection.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 2 2015, 2:02 PM
This revision was automatically updated to reflect the committed changes.