qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alan Conway" <acon...@redhat.com>
Subject Re: Review Request: Fix for bug 655078: Modified cluster_tests causes broker shut down with invalid-argument error.
Date Thu, 02 Dec 2010 16:15:56 GMT


> On 2010-12-02 04:33:13, Gordon Sim wrote:
> > Only comment is that cluster.cmake doesn't have the new UpdateDataExchange.cpp added
to it.

Thanks, will fix on trunk 


- Alan


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


On 2010-11-30 12:08:32, Alan Conway wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/130/
> -----------------------------------------------------------
> 
> (Updated 2010-11-30 12:08:32)
> 
> 
> Review request for qpid.
> 
> 
> Summary
> -------
> 
> This patch is a roll up of the following 3 patches:
> 
> commit 5b28517ab3890a7f5316f2c89964343b9cc0b0dd
> Author: Alan Conway <aconway@redhat.com>
> Date:   Mon Nov 29 14:39:46 2010 -0500
> 
>     Modified cluster_tests causes broker shut down with invalid-argument error.
> 
>     Described in https://bugzilla.redhat.com/show_bug.cgi?id=655078.  The
>     management agent's deleted-object list was not being replicated to new
>     members joining the cluster, so management generated fewer deleted
>     object notifications on the newer member, causing it to fail with an
>     invalid-argument error. The list is now being replicated correctly.
> 
> commit 733262b4d6cec8b0d30db949bfa93f11dc07773f
> Author: Alan Conway <aconway@redhat.com>
> Date:   Tue Nov 23 16:35:24 2010 -0500
> 
>     Add missing call to Message::setTimestamp in ManagementAgent::sendBufferLH.
> 
> commit 32ed120146db5f756d8ea4f9a7e0330bf5716f9d
> Author: Alan Conway <aconway@redhat.com>
> Date:   Tue Nov 23 15:50:00 2010 -0500
> 
>     Enable cluster-safe assertions on transition to CATCHUP
> 
>     Delaying until READY was causing multiple clientConnect management
>     events to be raised, because broker::Connection::setUserId relies on
>     sys::isCluster to avoid producing duplicate events with
> 
> 
> Diffs
> -----
> 
>   /trunk/qpid/cpp/src/cluster.mk 1040689 
>   /trunk/qpid/cpp/src/qpid/cluster/Cluster.cpp 1040689 
>   /trunk/qpid/cpp/src/qpid/cluster/Connection.h 1040689 
>   /trunk/qpid/cpp/src/qpid/cluster/Connection.cpp 1040689 
>   /trunk/qpid/cpp/src/qpid/cluster/UpdateClient.cpp 1040689 
>   /trunk/qpid/cpp/src/qpid/management/ManagementAgent.h 1040689 
>   /trunk/qpid/cpp/src/qpid/management/ManagementAgent.cpp 1040689 
>   /trunk/qpid/cpp/src/tests/cluster_tests.py 1040689 
>   /trunk/qpid/cpp/xml/cluster.xml 1040689 
> 
> Diff: https://reviews.apache.org/r/130/diff
> 
> 
> Testing
> -------
> 
> Passes make check, make check-long. Currently running make check-long in a loop.
> 
> 
> Thanks,
> 
> Alan
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message