ok, it's anyway a temporary thing for the sake of testing but I agree with
your suggestion, I'll change the property name.

Regards,
Tommaso

Il giorno mer 25 ott 2017 alle ore 15:38 Chetan Mehrotra <
chetan.mehro...@gmail.com> ha scritto:

> > +        String mmp = System.getProperty("cmmp");
>
> Better to use system property like "oak.lucene.cmmp"
> Chetan Mehrotra
>
>
> On Tue, Oct 24, 2017 at 9:56 PM,  <tomm...@apache.org> wrote:
> > Author: tommaso
> > Date: Tue Oct 24 16:26:05 2017
> > New Revision: 1813193
> >
> > URL: http://svn.apache.org/viewvc?rev=1813193&view=rev
> > Log:
> > OAK-6710 - possibly set mitigating merce policy via vm options
> >
> > Modified:
> >
>  
> jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexDefinition.java
> >
> > Modified:
> jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexDefinition.java
> > URL:
> http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexDefinition.java?rev=1813193&r1=1813192&r2=1813193&view=diff
> >
> ==============================================================================
> > ---
> jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexDefinition.java
> (original)
> > +++
> jackrabbit/oak/trunk/oak-lucene/src/main/java/org/apache/jackrabbit/oak/plugins/index/lucene/IndexDefinition.java
> Tue Oct 24 16:26:05 2017
> > @@ -1577,6 +1577,11 @@ public final class IndexDefinition imple
> >      }
> >
> >      private MergePolicy createMergePolicy() {
> > +        String mmp = System.getProperty("cmmp");
> > +        if (mmp != null) {
> > +            return new CommitMitigatingTieredMergePolicy();
> > +        }
> > +
> >          String mergePolicyName = getOptionalValue(definition,
> LuceneIndexConstants.MERGE_POLICY_NAME, null);
> >          MergePolicy mergePolicy = null;
> >          if (mergePolicyName != null) {
> >
> >
>

Reply via email to