ajantha-bhat commented on a change in pull request #3778:
URL: https://github.com/apache/carbondata/pull/3778#discussion_r473600323
##########
File path:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
##########
@@ -2400,6 +2400,23 @@ private CarbonCommonConstants() {
*/
public static final String CARBON_SI_SEGMENT_MERGE_DEFAULT = "false";
+ /**
+ * Until the threshold for complex filter is reached, row id will be set to
the bitset in
+ * implicit filter during secondary index pruning
+ */
+ public static final String SI_COMPLEX_FILTER_THRESHOLD =
"carbon.si.complex.filter.threshold";
+
+ /**
+ * Maximum value for complex filter threshold
+ */
+ public static final String SI_COMPLEX_FILTER_THRESHOLD_DEFAULT = "32000";
+
+ /**
+ * Property to decide if position id till row level or not
+ */
+ public static final String IS_TUPLE_ID_TILL_ROW_FOR_SI_COMPLEX =
Review comment:
In the community, we concluded that no need of row level position
reference. So, why this is required ?
----------------------------------------------------------------
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
