-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5529/
-----------------------------------------------------------

(Updated June 27, 2012, 10:13 p.m.)


Review request for qpid, Andrew Stitcher and Gordon Sim.


Changes
-------

Use abort instead of throwing


Description (updated)
-------

    A HA backup broker used to reject client connections by throwing out of 
ConnectionObserver::opened.
    This is not an error, but generates a lot of noise in the form of error log 
messages.
    This patch calls Connection::abort() instead of throwing, which does not 
leave any error log messages.


Diffs (updated)
-----

  /trunk/qpid/cpp/src/qpid/ha/BackupConnectionExcluder.h 1354717 
  /trunk/qpid/cpp/src/qpid/ha/ConnectionObserver.cpp 1354717 

Diff: https://reviews.apache.org/r/5529/diff/


Testing
-------

make check


Thanks,

Alan Conway

Reply via email to