chesnokoff commented on code in PR #12723:
URL: https://github.com/apache/ignite/pull/12723#discussion_r2815150939


##########
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/integration/SqlPlanHistoryIntegrationTest.java:
##########
@@ -551,6 +551,38 @@ public void testNoScanCountSuffixForUNION() throws 
Exception {
         checkMetrics(getSqlPlanHistory());
     }
 
+    /**
+     * Ensures H2 auto-generated numeric aliases (e.g. "_1", "_4") are 
normalized so that repeated executions of the same
+     * query produce a single unique plan in SQL plan history.
+     */
+    @Test
+    public void testH2AutoAliasInPlanHistory() throws Exception {
+        
assumeTrue(IndexingQueryEngineConfiguration.ENGINE_NAME.equals(sqlEngine));
+
+        assumeFalse(isPerfStatsEnabled);

Review Comment:
   PerfStat does not effect on logic of H2-plan normalization.
   
   Ok, let's remove this assumption.



-- 
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]

Reply via email to