Page MenuHomePhabricator
Differential D15378

Implement an iterator for build log chunks
ClosedPublic
Actions

Authored by epriestley on Mar 1 2016, 8:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 18, 1:01 AM
Unknown Object (File)
Sun, Dec 15, 2:30 AM
Unknown Object (File)
Tue, Dec 3, 8:38 AM
Unknown Object (File)
Wed, Nov 27, 7:26 AM
Unknown Object (File)
Tue, Nov 26, 6:52 PM
Unknown Object (File)
Mon, Nov 25, 9:58 AM
Unknown Object (File)
Thu, Nov 21, 4:02 PM
Unknown Object (File)
Nov 17 2024, 8:42 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T5822: Implement garbage collection / automatic archiving for Harbormaster logs
Commits
Restricted Diffusion Commit
rP6514237c0e01: Implement an iterator for build log chunks
Summary

Ref T5822. This will make it easier to compress and archive chunks without needing to hold them in memory.

Test Plan

Ran a build, looked at some logs.

Diff Detail

Repository
rP Phabricator
Branch
dry17
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/harbormaster/storage/build/HarbormasterBuildLog.php:190XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 10930
Build 13489: Run Core Tests
Build 13488: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 37073.Mar 1 2016, 8:26 PM
epriestley retitled this revision from to Implement an iterator for build log chunks.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T5822: Implement garbage collection / automatic archiving for Harbormaster logs.
chad accepted this revision.Mar 1 2016, 9:35 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 1 2016, 9:35 PM
Closed by commit rP6514237c0e01: Implement an iterator for build log chunks (authored by epriestley, committed by epriestley). · Explain WhyMar 1 2016, 11:25 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
2 lines
applications/
harbormaster/
storage/
build/
45 lines
23 lines
35 lines

Diff 37073

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/harbormaster/storage/build/HarbormasterBuildLog.php

Loading...
View Options

src/applications/harbormaster/storage/build/HarbormasterBuildLogChunk.php

Loading...
View Options

src/applications/harbormaster/storage/build/HarbormasterBuildLogChunkIterator.php

Loading...
Phabricator · Built by Phacility