zstan commented on code in PR #5311:
URL: https://github.com/apache/ignite-3/pull/5311#discussion_r1984546189
##########
modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItSecondaryIndexTest.java:
##########
@@ -965,23 +987,29 @@ public void testNullsInSearchRow() {
assertQuery("SELECT * FROM t WHERE i1 = ?")
.withParams(null)
- .matches(containsIndexScan("PUBLIC", "T", "T_IDX"))
+ .matches(containsTableScan("PUBLIC", "T"))
Review Comment:
i fix but I disagree here, as far as i remember such scope of tests
initially was planned for end-to-end index choosing tests. But i agree that for
now it all looks strange - after force statistic gathering it not acts as equal
behavior from user side ... so for consistency seems this class is not right
place for "testComplexIndexConditionInitialTableCreate" test.
--
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]
