On Wed, Sep 09, 2009 at 08:25:19AM -0400, Stephen Gallagher wrote: > On 09/09/2009 07:50 AM, Sumit Bose wrote: > > On Tue, Sep 08, 2009 at 08:32:55PM -0400, Stephen Gallagher wrote: > >> I have refactored nsssrv_cmd.c and created a new patch for the > >> EntryCacheNoWaitRefreshTimeout. > >> > >> I have created a new function, check_cache() which is a common entry > >> point for getpwnam, getpwuid, getgrnam and getgrgid to examine whether > >> the cache is still valid. > >> > >> Addressing other points from the review inline below. > >> > >> > >> On 08/17/2009 11:19 AM, Sumit Bose wrote: > >>> On Fri, Aug 14, 2009 at 03:46:54PM -0400, Stephen Gallagher wrote: > >>>> This timeout specifies the lifetime of a cache entry before it is > >>>> updated out-of-band. When this timeout is hit, the request will > >>>> still complete from cache, but the SSSD will also go and update > >>>> the cached entry in the background to extend the life of the > >>>> cache entry and reduce the wait time of a future request. > >>>> > >>>> Support for the EnumCacheNoWaitRefreshTimeout is still forthcoming, but > >>>> I wanted to get a formal review on this portion. > >>> > >>> > >>> NACK. I think this patch indicates that nsssrv_cmd.c needs some > >>> refactoring, please do this before adding more code. > >>> > >> > >> Done. > >> > > > > Works for me, but can you add a man page entry for > > EnumCacheNoWaitRefreshTimeout ? > > > > bye, > > Sumit > > _______________________________________________ > > sssd-devel mailing list > > sssd-devel@lists.fedorahosted.org > > https://fedorahosted.org/mailman/listinfo/sssd-devel > > Whoops, forgot to add those to the commit. New patch 0002 attached > (patch 0001 unaffected) > Thanks.
ACK bye, Sumit _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://fedorahosted.org/mailman/listinfo/sssd-devel