jpountz commented on a change in pull request #574: LUCENE-8292: Make TermsEnum 
fully abstract
URL: https://github.com/apache/lucene-solr/pull/574#discussion_r257244063
 
 

 ##########
 File path: lucene/core/src/java/org/apache/lucene/index/TermsEnum.java
 ##########
 @@ -271,4 +246,50 @@ public void seekExact(BytesRef term, TermState state) {
     }
 
   };
+
+  /**
+   * A base TermsEnum that adds default implementations for
+   * <ul>
+   *   <li>{@link #attributes()}</li>
+   *   <li>{@link #termState()}</li>
+   *   <li>{@link #seekExact(BytesRef)}</li>
+   *   <li>{@link #seekExact(BytesRef, TermState)}</li>
+   * </ul>
+   *
+   * In some cases, the default implementation may be slow and consume huge 
memory, so subclass SHOULD have its own
+   * implementation if possible.
+   */
+  public static abstract class BaseTermsEnum extends TermsEnum {
 
 Review comment:
   move to its own file?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to