We won't queue AMQP jobs
ClosedPublic
Actions

Authored by dereckson on Jan 9 2016, 20:24.

Tags

None

Referenced Files

	F3771099: D211.id500.diff
	Sat, Nov 23, 22:36

	F3770828: D211.id499.diff
	Sat, Nov 23, 21:37

	F3770370: D211.diff
	Sat, Nov 23, 20:04

	F3768342: D211.id499.diff
	Sat, Nov 23, 07:34

	Unknown Object (File)
	Fri, Nov 22, 19:21

	Unknown Object (File)
	Wed, Nov 13, 13:48

	Unknown Object (File)
	Sun, Nov 10, 13:16

	Unknown Object (File)
	Sun, Nov 10, 06:46

View All Files

Subscribers

None

Details

Reviewers

dereckson

Commits

Unknown Object (Diffusion Commit)
rNOTIF8a71e37cf6a9: We won't queue AMQP jobs
rNOTIFb1fd32a59030: We won't queue AMQP jobs

Summary

To put a message in a queue to put a message in another queue
won't give us any performance advantage.

So we can prune Laravel queue stuff from these classes.

Test Plan

Fire a notification.

Diff Detail

Repository

rNOTIF Notifications center

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 499.Jan 9 2016, 20:24

dereckson retitled this revision from to We won't queue AMQP jobs.

dereckson updated this object.

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

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Jan 9 2016, 20:24

dereckson edited edge metadata.

This revision is now accepted and ready to land.Jan 9 2016, 20:24

Closed by commit rNOTIFb1fd32a59030: We won't queue AMQP jobs (authored by dereckson, committed by dereckson). · Explain WhyJan 9 2016, 20:24

This revision was automatically updated to reflect the committed changes.

dereckson updated this object.Jan 9 2016, 20:30

dereckson edited edge metadata.

Revision Contents
Changeset List

Path

Size

app/

Listeners/

AMQPEventListener.php

3 lines

Diff 500

View Options

app/Listeners/AMQPEventListener.php

We won't queue AMQP jobsClosedPublicActions