Allow to set an exchange as durable
ClosedPublic
Actions

Authored by dereckson on Mar 12 2016, 05:49.

Details

Reviewers

dereckson

Maniphest Tasks

T781: Set the notifications exchange as durable

Commits

rKBROKER264c79c4695a: Allow to set an exchange as durable

Summary

The Nasqueron notifications center needs, as explained on T781,
to declare an exchange as durable.

This change allows so in AMQPBroker::setExchanteTarget.

This fixes the following AMQPProtocolChannelException:

PRECONDITION_FAILED - inequivalent arg 'durable' for exchange
'notifications' in vhost 'dev':
received 'false' but current is 'true'

Test Plan

Declare a durable exchange, when an exchange is already declared
in the broker.

Diff Detail

Repository

rKBROKER Keruald Broker

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson updated this revision to Diff 797.Mar 12 2016, 05:49

dereckson retitled this revision from to Allow to set an exchange as durable.

dereckson updated this object.

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

dereckson added a reviewer: dereckson.

dereckson accepted this revision.Mar 12 2016, 05:50

dereckson added a task: T781: Set the notifications exchange as durable.

dereckson edited edge metadata.

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

dereckson updated this object.Mar 12 2016, 05:51

dereckson edited edge metadata.

dereckson added a child revision: D337: Set the broker AMQP exchange as durable.Mar 12 2016, 05:53

Closed by commit rKBROKER264c79c4695a: Allow to set an exchange as durable (authored by dereckson, committed by dereckson). · Explain WhyMar 12 2016, 05:56

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

AMQPBroker.php

6 lines

Diff 799

View Options

Allow to set an exchange as durableClosedPublicActions