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
Unknown Object (File)
Mon, Dec 30, 3:38 AM
Unknown Object (File)
Wed, Dec 25, 7:21 PM
Unknown Object (File)
Tue, Dec 24, 2:39 AM
Unknown Object (File)
Dec 12 2024, 5:27 AM
Unknown Object (File)
Dec 7 2024, 9:25 PM
Unknown Object (File)
Dec 3 2024, 5:35 PM
Unknown Object (File)
Nov 20 2024, 12:33 PM
Unknown Object (File)
Nov 20 2024, 12:33 PM
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
Branch
dx5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8140
Build 9296: Run Core Tests
Build 9295: arc lint + arc unit

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.