siddharthteotia commented on a change in pull request #5503: URL: https://github.com/apache/incubator-pinot/pull/5503#discussion_r436197601
########## File path: pinot-spi/src/main/java/org/apache/pinot/spi/config/table/FieldConfig.java ########## @@ -33,15 +33,16 @@ private final IndexType _indexType; private final Map<String, String> _properties; - public static String BLOOM_FILTER_COLUMN_KEY = "bloom.filter"; Review comment: I haven't yet done the migration from IndexingConfig to FieldConfig. It is only being used for text. So this change doesn't affect anyone. Just wanted to clean up names and remove the dotted notation. Going to migrate to FieldConfig soon ---------------------------------------------------------------- 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