Re: Patch missing in 4.4-forward

2014-05-19 Thread Yoshikazu Nojima
Gaurav, This patch is intended to be released in 4.5 release, because when I wrote up it, 4.4 feature freeze date was already passed. I'm not sure which unit test are you talking about? Regards, Noji 2014-05-19 8:20 GMT-06:00 Gaurav Aradhye : > Hi, > > The patch related to feature "Volume Prov

Re: Patch missing in 4.4-forward

2014-05-19 Thread Gaurav Aradhye
Hello Yoshikazu, Please take a look at this patch (https://reviews.apache.org/r/21682/). These are the unit tests I am talking about. These should not be present in 4.4-forward right? Regards, Gaurav On Mon, May 19, 2014 at 8:28 PM, Yoshikazu Nojima wrote: > Gaurav, > > This patch is intended

Re: Patch missing in 4.4-forward

2014-05-20 Thread Daan Hoogland
Gaurav, to avoid confusion: these look like integration tests. Is that what you mean? On Tue, May 20, 2014 at 7:17 AM, Gaurav Aradhye wrote: > Hello Yoshikazu, > > Please take a look at this patch (https://reviews.apache.org/r/21682/). > These are the unit tests I am talking about. These should n

Re: Patch missing in 4.4-forward

2014-05-20 Thread Gaurav Aradhye
Yes. Integration smoke test. Regards, Gaurav On Tue, May 20, 2014 at 1:52 PM, Daan Hoogland wrote: > Gaurav, to avoid confusion: these look like integration tests. Is that > what you mean? > > On Tue, May 20, 2014 at 7:17 AM, Gaurav Aradhye > wrote: > > Hello Yoshikazu, > > > > Please take a l

Re: Patch missing in 4.4-forward

2014-05-20 Thread Min Chen
I ran into same issues in analyzing automation test. The related integration tests are merged into 4.4-forward along with marvin test framework merge, but this patch does not exist in 4.4-forward, so we should either disable these two integration tests, or pull that patch code into 4.4-forward. Th

RE: Patch missing in 4.4-forward

2014-05-20 Thread Santhosh Edukulla
missing in 4.4-forward I ran into same issues in analyzing automation test. The related integration tests are merged into 4.4-forward along with marvin test framework merge, but this patch does not exist in 4.4-forward, so we should either disable these two integration tests, or pull that patch code