Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

uot;Jeff Darcy" <jda...@redhat.com> > > To: "Niels de Vos" <nde...@redhat.com>, "Ira Cooper" <i...@redhat.com> > > Cc: "Gluster Devel" <gluster-devel@gluster.org> > > Sent: Saturday, December 5, 2015 12:04:29 AM > > Subje

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

ff Darcy" <jda...@redhat.com> > To: "Niels de Vos" <nde...@redhat.com>, "Ira Cooper" <i...@redhat.com> > Cc: "Gluster Devel" <gluster-devel@gluster.org> > Sent: Saturday, December 5, 2015 12:04:29 AM > Subject: Re: [Gluster-d

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On Thu, Dec 03, 2015 at 06:34:56PM -0500, Ira Cooper wrote: > Kaleb KEITHLEY writes: > > > On 12/03/2015 03:10 PM, Ira Cooper wrote: > >> Poornima Gurusiddaiah writes: > >> > >>> Hi, > >>> > >>> Brief Background: > >>> > >>> For the

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On Fri, Dec 04, 2015 at 07:51:41AM -0500, Ira Cooper wrote: > Niels de Vos writes: > > > On Thu, Dec 03, 2015 at 06:34:56PM -0500, Ira Cooper wrote: > >> Kaleb KEITHLEY writes: > >> > >> > On 12/03/2015 03:10 PM, Ira Cooper wrote: > >> >> Poornima

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On 12/04/2015 07:51 AM, Ira Cooper wrote: > why not use storage in the client_t instead of thread local? >>> >>> It comes down to the use case. For Samba, the right spot is almost >>> certainly the fd, because lease keys are a per-handle (which we map to >>> fd) property. >>> >>>

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

Niels de Vos writes: > On Thu, Dec 03, 2015 at 06:34:56PM -0500, Ira Cooper wrote: >> Kaleb KEITHLEY writes: >> >> > On 12/03/2015 03:10 PM, Ira Cooper wrote: >> >> Poornima Gurusiddaiah writes: >> >> >> >>> Hi, >> >>> >> >>>

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On December 4, 2015 at 8:25:10 AM, Niels de Vos (nde...@redhat.com) wrote: > Okay, so you meant to say that client_t "is a horror show" for this > particular use-case (lease_id). It indeed does not sound suitable to use > client_t here. > > I'm not much of a fan for using Thread Local Storage

[Gluster-devel] libgfapi changes to add lk_owner and lease ID

Hi, Brief Background: For the below two features, we need ligfapi to take 2 other parameters from the applications for most number of fops. http://review.gluster.org/#/c/12014/ http://review.gluster.org/#/c/11980/ For leases to work as explained in the above doc, every file

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On 12/03/2015 03:10 PM, Ira Cooper wrote: > Poornima Gurusiddaiah writes: > >> Hi, >> >> Brief Background: >> >> For the below two features, we need ligfapi to take 2 other parameters from >> the applications for most number of fops. >>

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

On Thu, Dec 03, 2015 at 09:11:34AM -0500, Poornima Gurusiddaiah wrote: > Hi, > > Brief Background: > > For the below two features, we need ligfapi to take 2 other parameters from > the applications for most number of fops. > http://review.gluster.org/#/c/12014/ >

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

Poornima Gurusiddaiah writes: > Hi, > > Brief Background: > > For the below two features, we need ligfapi to take 2 other parameters from > the applications for most number of fops. > http://review.gluster.org/#/c/12014/ >

Re: [Gluster-devel] libgfapi changes to add lk_owner and lease ID

Kaleb KEITHLEY writes: > On 12/03/2015 03:10 PM, Ira Cooper wrote: >> Poornima Gurusiddaiah writes: >> >>> Hi, >>> >>> Brief Background: >>> >>> For the below two features, we need ligfapi to take 2 other parameters from >>> the