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