incubator-etch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From scott comer <>
Subject Re: [vote] include the fix for etch-25 in release 1.0.2
Date Tue, 20 Jan 2009 15:47:17 GMT
i do not see any negatives or questions on this vote, there are three 
for including the fix, me, james, and jd. so be it.

scott out

scott comer (sccomer) wrote:
> DefaultDeliveryService ends call with mb.CloseDelivery() and it should 
> end it with mb.CloseRead()
> -------------------------------------------------------------------------------------------------

>                 Key: ETCH-25
>                 URL:
>             Project: Etch
>          Issue Type: Bug
>          Components: csharp-binding, java-binding
>    Affects Versions: 1.0.1, 1.0.0
>            Reporter: scott comer
> when done reading the mailbox, the right method to call is 
> CloseRead(). somehow the code actually calls CloseDelivery(), a method 
> used to shutdown the writing side of the mailbox. CloseRead() calls 
> CloseDelivery() first off, but then takes the additional steps of 
> redelivering any messages left in the mailbox.
> The distinction is rather fine, and this bug was found by reading the 
> source code. There are certain ways you could exercise this bug, but 
> it would be very rare and not have any real impact. So, it isn't an 
> emergency to fix it, but it is also a relatively benign fix. I've 
> already made the change in my tree and tested it. The patch is attached.
> ------------------
> there is little risk. all unit tests have passed.
> scott out

View raw message