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

	F18847722: D13451.id32554.diff
	Thu, Oct 30, 1:51 AM

	F18844838: D13451.diff
	Wed, Oct 29, 7:48 AM

	F18807760: D13451.id.diff
	Oct 19 2025, 3:25 AM

	F18804206: D13451.diff
	Oct 18 2025, 6:50 AM

	F18673597: D13451.id32555.diff
	Sep 25 2025, 11:17 AM

	F18672507: D13451.id.diff
	Sep 25 2025, 7:38 AM

	F18672505: D13451.diff
	Sep 25 2025, 7:38 AM

	F18609568: D13451.id.diff
	Sep 14 2025, 3:24 AM

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