I feel that this change should not be included in the 0.10 release. I think we are too close to release (RC1 is due tomorrow yes?) and that there isnt significant enough testing in this area to provide enough confidence when making what is actually an appreciable change in client behaviour at this stage.
So far as I can tell from the quick look I've taken so far AddressBasedDestinationTest is the only test which exercises the Addressing code to any extent, and does so often by prefixing the address strings with 'ADDR:' because the test profiles are all running set to use the old BindingURL syntax. As a result it would seem we ultimately arent testing the precise execution paths for many operations users will undertake at all. Robbie On 14 March 2011 16:48, Rajith Attapattu <[email protected]> wrote: > The attached patch in QPID-3143 contains a simple one line fix and a > modified test case to test the change. > The same is committed to trunk at rev 1081460. > The fix is low risk IMHO. > > However the fix is important as it makes the session.createQueue method > spec > complaint. > It also allows the default behaviour to be easily overridden if the user > chooses do so. > > Regards, > > Rajith >
