Do all push mail in a bunch at the end, too
ClosedPublic
Actions

Authored by epriestley on Jun 25 2015, 10:54 PM.

Tags

None

Referenced Files

	F15420173: D13451.id32554.diff
	Fri, Mar 21, 11:57 AM

	F15419366: D13451.diff
	Fri, Mar 21, 5:36 AM

	F15375997: D13451.diff
	Thu, Mar 13, 1:09 AM

	F15375225: D13451.id.diff
	Wed, Mar 12, 10:12 PM

	Unknown Object (File)
	Sun, Mar 2, 5:13 AM

	Unknown Object (File)
	Feb 8 2025, 3:57 PM

	Unknown Object (File)
	Feb 8 2025, 3:57 PM

	Unknown Object (File)
	Feb 8 2025, 3:56 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rP14229042a200: Do all push mail in a bunch at the end, too

Summary

Same deal as transaction mail. Improves robustness.

Test Plan

Pushed stuff.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 32554.Jun 25 2015, 10:54 PM

epriestley retitled this revision from to Do all push mail in a bunch at the end, too.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptJun 25 2015, 10:54 PM

btrahan accepted this revision.Jun 25 2015, 10:54 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Jun 25 2015, 10:54 PM

Closed by commit rP14229042a200: Do all push mail in a bunch at the end, too (authored by epriestley, committed by epriestley). · Explain WhyJun 25 2015, 10:55 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

worker/

PhabricatorRepositoryPushMailWorker.php

10 lines

Diff 32555

View Options

src/applications/repository/worker/PhabricatorRepositoryPushMailWorker.php

Do all push mail in a bunch at the end, tooClosedPublicActions