Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-6879

Allow to define custom CharTokenizer using Java 8 Lambdas/Method references

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 6.0
    • 6.0
    • modules/analysis
    • New

    Description

      As a followup from LUCENE-6874, I thought about how to generate custom CharTokenizers wthout subclassing. I had this quite often and I was a bit annoyed, that you had to create a subclass every time.

      This issue is using the pattern like ThreadLocal or many collection methods in Java 8: You have the (abstract) base class and you define a factory method named fromXxxPredicate (like ThreadLocal.withInitial(() -> value).

      public static CharTokenizer fromTokenCharPredicate(java.util.function.IntPredicate predicate)
      

      This would allow to define a new CharTokenizer with a single line statement using any predicate:

      // long variant with lambda:
      Tokenizer tok = CharTokenizer.fromTokenCharPredicate(c -> !UCharacter.isUWhiteSpace(c));
      
      // method reference for separator char predicate + normalization by uppercasing:
      Tokenizer tok = CharTokenizer.fromSeparatorCharPredicate(UCharacter::isUWhiteSpace, Character::toUpperCase);
      
      // method reference to custom function:
      private boolean myTestFunction(int c) {
       return (cracy condition);
      }
      Tokenizer tok = CharTokenizer.fromTokenCharPredicate(this::myTestFunction);
      

      I know this would not help Solr users that want to define the Tokenizer in a config file, but for real Lucene users this Java 8-like way would be easy and elegant to use. It is fast as hell, as it is just a reference to a method and Java 8 is optimized for that.

      The inverted factories fromSeparatorCharPredicate() are provided to allow quick definition without lambdas using method references. In lots of cases, like WhitespaceTokenizer, predicates are on the separator chars (isWhitespace(int), so using the 2nd set of factories you can define them without the counter-intuitive negation. Internally it just uses Predicate#negate().

      The factories also allow to give the normalization function, e.g. to Lowercase, you may just give Character::toLowerCase as IntUnaryOperator reference.

      Attachments

        1. LUCENE-6879.patch
          11 kB
          Uwe Schindler
        2. LUCENE-6879.patch
          7 kB
          Uwe Schindler

        Issue Links

          Activity

            People

              uschindler Uwe Schindler
              uschindler Uwe Schindler
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: