Re: (ITS#8584) reloadhint for syncprov overlay should default to TRUE

[quanah@symas.com]

> On Feb 7, 2017, at 10:25 AM, hyc@symas.com wrote:
>=20
> Quanah Gibson-Mount wrote:
>> --On Tuesday, February 07, 2017 8:53 AM +0000 Howard Chu <hyc@symas.com> w=
rote:
>>=20
>>=20
>>> The relevant part of the doc is "It must be set TRUE when using the
>>> accesslog overlay for delta-based syncrepl support." Otherwise it has no=

>>> effect, so there is no reason to change it in the default case.
>>=20
>> Actually, what the manpage says is:
>>=20
>> This option should be set TRUE when working with
>> newer releases that properly support this flag. It must  be  set
>> TRUE  when  using the accesslog overlay for delta-based syncrepl
>> replication support.
>>=20
>> So, that clearly states it should *always* be TRUE unless you have a 2.3.=
11 or
>> earlier consumer, which is technically not allowable.  There is zero reas=
on
>> for this tunable to exist.  It should be deprecated and the internal code=

>> changed to simply set the flag to TRUE.  One less item for anyone to have=
 to
>> ever deal with is a win.
>=20
> "when working with newer releases that properly support this flag."
>=20
> The syncrepl consumer only pays attention to this flag in delta-sync. Ther=
e is=20
> no reason to use reloadhint in regular syncrepl. Turning it on by default o=
nly=20
> slows down the refresh negotiation in regular syncrepl.
>=20
> --=20
>   -- Howard Chu
>   CTO, Symas Corp.           http://www.symas.com
>   Director, Highland Sun     http://highlandsun.com/hyc/
>   Chief Architect, OpenLDAP  http://www.openldap.org/project/
>=20

Then the man page needs updating since the information on it is blatantly wr=
ong.  The "newer releases" is clearly indicated to be anything later than 2.=
3.11.

--Quanah=