gluster-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Gluster-devel] [RFC] core: Introduce internal fop types.


From: Raghavendra Gowdappa
Subject: Re: [Gluster-devel] [RFC] core: Introduce internal fop types.
Date: Fri, 3 Jan 2014 04:24:25 -0500 (EST)


----- Original Message -----
> From: "Vijay Bellur" <address@hidden>
> To: "Raghavendra Gowdappa" <address@hidden>, "Gluster Devel" <address@hidden>
> Sent: Thursday, January 2, 2014 7:41:58 PM
> Subject: Re: [Gluster-devel] [RFC] core: Introduce internal fop types.
> 
> On 12/13/2013 10:09 AM, Raghavendra Gowdappa wrote:
> > Hi all,
> >
> > It is increasingly becoming a necessity that xlators other than dht/afr to
> > be aware of internal fops (fops not sent by kernel). Some of the xlators
> > that use internal fops are:
> >
> > 1. changelog - to ignore link/unlink (triggered as part of rename) on
> > dht-linkfiles
> > 2. posix -  to overload mknod call to act as a link if file already exists
> > (consumers - afr and gfid-acess)
> >
> > We are seeing similar needs in quota and marker:
> >
> > 1. Quota-enforcer: To treat link/unlinks on dht-datafiles triggered as part
> > of rename equivalent to renames [1]
> > 2. Marker: To not account dht-linkfiles
> >
> > Couple of months back when working on quota, there was a consensus that
> > GLUSTERFS_INTERNAL_FOP_KEY is overloaded and should be cleaned up. Hence,
> > I am proposing to introduce internal-fop types [2]. Please let me know
> > your opinions.
> >
> > [1] http://review.gluster.com/#/c/6399/
> > [2] http://review.gluster.org/#/c/6497/
> >
> 
> I think having more apt keys for performing relevant actions is good to
> have. While this change is performed, we need to ensure that the
> behavior of posix/changelog other consumers remain backward compatible.

Ok. I'll go ahead with this change keeping backward compatibility in mind.

> 
> -Vijay
> 
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]