Paths

Table of Contentst

Differential D15464

Fix a minor issue in Nuance which could cause the trigger daemon to poll too often
ClosedPublic
Actions

Authored by epriestley on Mar 11 2016, 11:47 PM.

Tags

None

Referenced Files

	F15544725: D15464.id.diff
	Sat, Apr 26, 6:46 AM

	F15540089: D15464.diff
	Fri, Apr 25, 6:31 AM

	F15514336: D15464.id.diff
	Fri, Apr 18, 2:06 AM

	F15448263: D15464.id37269.diff
	Fri, Mar 28, 3:47 AM

	F15439462: D15464.id.diff
	Mar 26 2025, 7:38 AM

	F15439020: D15464.id37267.diff
	Mar 26 2025, 5:15 AM

	F15434720: D15464.diff
	Mar 25 2025, 4:59 AM

	F15421503: D15464.id.diff
	Mar 22 2025, 12:23 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10537: Nuance Infrastructure

Commits

Restricted Diffusion Commit
rPde23ba0002f8: Fix a minor issue in Nuance which could cause the trigger daemon to poll too…

Summary

Ref T10537. Currently, when you have at least two cursors, the daemon can poll too frequently when processing the last source because it never hits the end-of-list condition.

Test Plan

Ran bin/phd debug trigger.
Observed huge volumes of output before change as triggers fired as fast as possible.
Observed reasonable poll frequency after change.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37267.Mar 11 2016, 11:47 PM

epriestley retitled this revision from to Fix a minor issue in Nuance which could cause the trigger daemon to poll too often.

epriestley updated this object.

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

epriestley added a reviewer: chad.

epriestley added a task: T10537: Nuance Infrastructure.

Herald edited edge metadata. · View Herald TranscriptMar 11 2016, 11:47 PM

chad accepted this revision.Mar 12 2016, 12:05 AM

chad edited edge metadata.

This revision is now accepted and ready to land.Mar 12 2016, 12:05 AM

Closed by commit rPde23ba0002f8: Fix a minor issue in Nuance which could cause the trigger daemon to poll too… (authored by epriestley, committed by epriestley). · Explain WhyMar 12 2016, 1:04 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

daemon/

workers/

PhabricatorTriggerDaemon.php

2 lines

Diff 37269

src/infrastructure/daemon/workers/PhabricatorTriggerDaemon.php

Loading...