----- Original Message ----- > From: "Atin Mukherjee" <[email protected]> > To: "Niels de Vos" <[email protected]>, "Vijaikumar M" <[email protected]> > Cc: "Raghavendra G" <[email protected]>, "Gluster Devel" > <[email protected]> > Sent: Wednesday, June 24, 2015 10:15:12 AM > Subject: Re: [Gluster-devel] /tests/bugs/quota/bug-1153964.t is consistently > failing > > When is this patch getting merged, this is blocking other patches to get in.
Revert of http://review.gluster.org/11311, is waiting for regression runs to pass. There are three patches (duplicates of each other). If anyone of them pass both regression runs, I'll merge them. As far as refcounting mechanism go, it'll take some time to review and merge the patch. > > ~Atin > > On 06/23/2015 06:26 PM, Niels de Vos wrote: > > On Tue, Jun 23, 2015 at 05:30:39PM +0530, Vijaikumar M wrote: > >> > >> > >> On Tuesday 23 June 2015 04:28 PM, Niels de Vos wrote: > >>> On Tue, Jun 23, 2015 at 03:45:43PM +0530, Vijaikumar M wrote: > >>>> I have submitted below patch which fixes this issue. I am handling > >>>> memory > >>>> clean-up with reference countmechanism. > >>>> > >>>> http://review.gluster.org/#/c/11361 > >>> Is there a reason you can not use the (new) refcounting functions that > >>> were introduceed with http://review.gluster.org/11022 ? > >> > >> I was not aware that ref-counting patch was merged. Sure we will use these > >> function and re-submit my patch. > > > > Ok, thanks! > > Niels > > > >> > >> Thanks, > >> Vijay > >> > >> > >>> It would be nicer to standardize all refcounting mechanisms on one > >>> implementation. I hope we can replace existing refcounting with this one > >>> too. Introducing more refcounting ways is not going to be helpful. > >>> > >>> Thanks, > >>> Niels > >>> > >>>> Thanks, > >>>> Vijay > >>>> > >>>> > >>>> > >>>> > >>>> On Tuesday 23 June 2015 12:58 PM, Raghavendra G wrote: > >>>>> Multiple replies to same query. Pick one ;). > >>>>> > >>>>> On Tue, Jun 23, 2015 at 12:55 PM, Venky Shankar > >>>>> <[email protected] > >>>>> <mailto:[email protected]>> wrote: > >>>>> > >>>>> OK. Two reverts of the same patch ;) > >>>>> > >>>>> Pick one. > >>>>> > >>>>> On Tue, Jun 23, 2015 at 12:51 PM, Raghavendra Gowdappa > >>>>> <[email protected] <mailto:[email protected]>> wrote: > >>>>> > Seems like its a memory corruption caused by: > >>>>> > http://review.gluster.org/11311 > >>>>> > > >>>>> > I've reverted the patch at: > >>>>> > http://review.gluster.org/11360 > >>>>> > > >>>>> > ----- Original Message ----- > >>>>> >> From: "Xavier Hernandez" <[email protected] > >>>>> <mailto:[email protected]>> > >>>>> >> To: "Gluster Devel" <[email protected] > >>>>> <mailto:[email protected]>> > >>>>> >> Sent: Tuesday, June 23, 2015 12:44:47 PM > >>>>> >> Subject: [Gluster-devel] /tests/bugs/quota/bug-1153964.t is > >>>>> consistently failing > >>>>> >> > >>>>> >> Hi, > >>>>> >> > >>>>> >> the quota test bug-1153964.t is failing consistently for a > >>>>> >> totally > >>>>> >> unrelated patch. Is this a known issue ? > >>>>> >> > >>>>> >> > >>>>> > >>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11142/consoleFull > >>>>> >> > >>>>> > >>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11165/consoleFull > >>>>> >> > >>>>> > >>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11172/consoleFull > >>>>> >> > >>>>> > >>>>> http://build.gluster.org/job/rackspace-regression-2GB-triggered/11191/consoleFull > >>>>> >> > >>>>> >> Xavi > >>>>> >> _______________________________________________ > >>>>> >> Gluster-devel mailing list > >>>>> >> [email protected] <mailto:[email protected]> > >>>>> >> http://www.gluster.org/mailman/listinfo/gluster-devel > >>>>> >> > >>>>> > _______________________________________________ > >>>>> > Gluster-devel mailing list > >>>>> > [email protected] <mailto:[email protected]> > >>>>> > http://www.gluster.org/mailman/listinfo/gluster-devel > >>>>> _______________________________________________ > >>>>> Gluster-devel mailing list > >>>>> [email protected] <mailto:[email protected]> > >>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > >>>>> > >>>>> > >>>>> > >>>>> > >>>>> -- > >>>>> Raghavendra G > >>>>> > >>>>> > >>>>> _______________________________________________ > >>>>> Gluster-devel mailing list > >>>>> [email protected] > >>>>> http://www.gluster.org/mailman/listinfo/gluster-devel > >>>> _______________________________________________ > >>>> Gluster-devel mailing list > >>>> [email protected] > >>>> http://www.gluster.org/mailman/listinfo/gluster-devel > >> > > _______________________________________________ > > Gluster-devel mailing list > > [email protected] > > http://www.gluster.org/mailman/listinfo/gluster-devel > > > > -- > ~Atin > _______________________________________________ > Gluster-devel mailing list > [email protected] > http://www.gluster.org/mailman/listinfo/gluster-devel > _______________________________________________ Gluster-devel mailing list [email protected] http://www.gluster.org/mailman/listinfo/gluster-devel
