You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@directory.apache.org by Emmanuel Lecharny <el...@gmail.com> on 2010/05/29 10:35:14 UTC
ChangeLog enabled by default ?
Hi guys,
the ChangeLog interceptor seems to be enabled by default when using ADS
embedded, and is disabled when running the server standalone.
IMO, it should be disabled by default, unless we explicitly enable it or
when running unit tests.
The reason why it should be disabled is that it's a very costly
interceptor useful only for tests (or special cases).
I suggest that we disable it by default.
thoughts ?
--
Regards,
Cordialement,
Emmanuel Lécharny
www.nextury.com
Re: ChangeLog enabled by default ?
Posted by Alex Karasulu <ak...@gmail.com>.
+1
Sent from my iPhone
On May 29, 2010, at 11:35 AM, Emmanuel Lecharny <el...@gmail.com>
wrote:
> Hi guys,
>
> the ChangeLog interceptor seems to be enabled by default when using
> ADS embedded, and is disabled when running the server standalone.
>
> IMO, it should be disabled by default, unless we explicitly enable
> it or when running unit tests.
>
> The reason why it should be disabled is that it's a very costly
> interceptor useful only for tests (or special cases).
>
> I suggest that we disable it by default.
>
> thoughts ?
>
> --
> Regards,
> Cordialement,
> Emmanuel Lécharny
> www.nextury.com
>
>
Re: ChangeLog enabled by default ?
Posted by Pierre-Arnaud Marcelot <pa...@marcelot.net>.
+1
You're absolutely right.
Regards,
Pierre-Arnaud
On 29 mai 2010, at 10:35, Emmanuel Lecharny wrote:
> Hi guys,
>
> the ChangeLog interceptor seems to be enabled by default when using ADS embedded, and is disabled when running the server standalone.
>
> IMO, it should be disabled by default, unless we explicitly enable it or when running unit tests.
>
> The reason why it should be disabled is that it's a very costly interceptor useful only for tests (or special cases).
>
> I suggest that we disable it by default.
>
> thoughts ?
>
> --
> Regards,
> Cordialement,
> Emmanuel Lécharny
> www.nextury.com
>
>