kadirozde commented on code in PR #2428:
URL: https://github.com/apache/phoenix/pull/2428#discussion_r3192154208
##########
phoenix-core-client/src/main/java/org/apache/phoenix/query/QueryServicesOptions.java:
##########
@@ -214,6 +216,9 @@ public class QueryServicesOptions {
public static final boolean DEFAULT_IS_NAMESPACE_MAPPING_ENABLED = false;
public static final boolean DEFAULT_IS_SYSTEM_TABLE_MAPPED_TO_NAMESPACE =
true;
public static final int DEFAULT_MAX_IN_LIST_SKIP_SCAN_SIZE = 50000;
+ public static final boolean DEFAULT_WHERE_OPTIMIZER_V2_ENABLED = true;
Review Comment:
The PR updates a large number of existing test assertions to match V2's
behavior rather than keeping the V1 expected output — that's fine when the V2
output is objectively better (tighter scans, correct slotSpan). These do not
introduce data integrity issues.
--
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]
