npawar commented on code in PR #8601:
URL: https://github.com/apache/pinot/pull/8601#discussion_r863329647
##########
pinot-spi/src/main/java/org/apache/pinot/spi/config/table/IndexingConfig.java:
##########
@@ -64,6 +64,7 @@ public class IndexingConfig extends BaseJsonConfig {
// TODO: Add a new configuration related to the segment generation
private boolean _autoGeneratedInvertedIndex;
private boolean _createInvertedIndexDuringSegmentGeneration;
+ private boolean _createBloomFilterDuringSegmentGeneration;
Review Comment:
So you're suggesting we change this default behavior, and call it out in
release notes as backward-incompat?
That sounds good. If we're doing this, then might as well also (in separate
PR)
1. deprecate inv index field, tag the PR as release-notes
2. call out in next release it's deprecated
3. in future release remove it
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
