siddharthteotia commented on a change in pull request #5667:
URL: https://github.com/apache/incubator-pinot/pull/5667#discussion_r451354267



##########
File path: pinot-core/src/main/java/org/apache/pinot/core/common/DataSource.java
##########
@@ -60,6 +60,13 @@
   @Nullable
   InvertedIndexReader<?> getRangeIndex();
 
+  /**
+   * Returns the text index for the column if exists, or {@code null} if not.
+   * <p>TODO: Have a separate interface for text index.
+   */
+  @Nullable
+  InvertedIndexReader<?> getTextIndex();

Review comment:
       I'd like to do the cleanup for TextIndex in this PR itself. Both text 
and range followed the InvertedIndexReader interface.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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



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

Reply via email to