[SSSD] [PATCH] confdb: warn if memcache_timeout > than entry_cache

Pavel Brezina pbrezina at redhat.com
Tue Sep 22 20:50:02 UTC 2015



----- Original Message -----
> From: "Pavel Reichl" <preichl at redhat.com>
> To: sssd-devel at lists.fedorahosted.org
> Sent: Tuesday, September 22, 2015 1:31:45 PM
> Subject: Re: [SSSD] [PATCH] confdb: warn if memcache_timeout > than entry_cache
> 
> 
> 
> On 09/22/2015 10:25 AM, Pavel Březina wrote:
> > On 09/21/2015 04:33 PM, Pavel Reichl wrote:
> >> Hello, please see simple patch attached.
> >
> > Hi,
> > the patch looks good in general. However, I'd like to rephrase the warning.
> >
> >> +              "%s is lesser then %s, sysdb will not be updated for
> >> users.\n",
> >                          ^lower
> >
> > How about:
> >
> > %s is lower then %s. User records will not be updated before memory cache
> > entry expires.
> >
> > %s is lower then %s. User records will not be updated before memory cache
> > entry gets expired.
> > _______________________________________________
> 
> Hello Pavel, thanks for comment. I updated the patch as you proposed.
> 
> I did quick research on usage of smaller/lower/lesser and all seem to for
> correct for our case, but I agree that lower might sound more common.

Hi,
I am not a native speaker but lesser sounds quite strange in this sentence. But I admit it may be my personal opinion and it can be grammatically correct (smaller sounds good to me as well).

On the other hand the first sentence I proposed and that you chose for your patch (entry expires) seems strange to me as well and I suggested it anyway. I would personally pick "entry gets expired/becomes expired/is expired" but I think "expires" is grammatically correct here and it is shorter. Any native speaker wants to chime in here? :-)


More information about the sssd-devel mailing list