[GitHub] [phoenix] stoty commented on a diff in pull request #1631: Branch for testinf Phoenix-6984, phoenix-6983 and phoenix 6959 together on 5.1

["stoty (via GitHub)" <gi...@apache.org>]

stoty commented on code in PR #1631:
URL: https://github.com/apache/phoenix/pull/1631#discussion_r1235668080


##########
phoenix-core/src/main/java/org/apache/phoenix/optimize/QueryOptimizer.java:
##########
@@ -369,6 +384,10 @@ private QueryPlan addPlan(PhoenixStatement statement, SelectStatement select, PT
                  * otherwise we just don't use this index (as opposed to trying to join back from
                  * the index table to the data table.
                  */
+                // Reset the state changes from the attempt above
+                indexTableRef.setHinted(false);

Review Comment:
   We'd use the skip-scan-join plan with the projector for the server merge plan, which doesn't work.



-- 
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: issues-unsubscribe@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org