Re: Request for inclusion into the 0.22 branch

2013-04-01 Thread Robbie Gemmell
FWIW, while I'd like to see the additional testing done first I'd much rather the second change went into RC1 without the test changes than leave it the way it is on the 0.22 branch just now following the original change. Robbie On 27 Mar 2013 21:31, Justin Ross justin.r...@gmail.com wrote: Hi,

Request for inclusion into the 0.22 branch

2013-03-27 Thread Rajith Attapattu
Hey Justin, I want to include the following commits into the 0.22 branch. http://svn.apache.org/r1461324 http://svn.apache.org/r1461329 Once I make some adjustments to the tests, I would need to port that as well. I will send a follow up email when I've got that sorted out. Regards, Rajith

Re: Request for inclusion into the 0.22 branch

2013-03-27 Thread Rajith Attapattu
Sorry I forgot to include the JIRA in the email. Please see https://issues.apache.org/jira/browse/QPID-3769 for context. Rajith On Wed, Mar 27, 2013 at 12:00 PM, Rajith Attapattu rajit...@gmail.com wrote: Hey Justin, I want to include the following commits into the 0.22 branch.

Re: Request for inclusion into the 0.22 branch

2013-03-27 Thread Justin Ross
Hi, Rajith. I read the jira comments, and it looks like a little more work is required before we put it on the release branch (the testing for the jndi case). On Wed, Mar 27, 2013 at 12:04 PM, Rajith Attapattu rajit...@gmail.com wrote: Sorry I forgot to include the JIRA in the email. Please

Re: Request for inclusion into the 0.22 branch

2013-03-27 Thread Rajith Attapattu
On Wed, Mar 27, 2013 at 5:31 PM, Justin Ross justin.r...@gmail.com wrote: Hi, Rajith. I read the jira comments, and it looks like a little more work is required before we put it on the release branch (the testing for the jndi case). Indeed, this review request is for the whole package :P I