[ https://issues.apache.org/jira/browse/CASSANDRA-19034?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17788753#comment-17788753 ]
Andres de la Peña commented on CASSANDRA-19034: ----------------------------------------------- [~mike_tr_adamson] the patch in CASSANDRA-19040 is mostly ready. Even if the big refactor in CASSANDRA-18166 solves the issue in {{testContainsKeyAndContainsWithIndexOnMapValue}}, I'd like to commit it anyway because it adds testing around combinations of {{CONTAINS}}/{{CONTAINS KEY}} and filtering. The bug fixing part is a single-line change that surely won't get in the way of CASSANDRA-19040. wdyt? > SelectTest fails when run with SAI index > ---------------------------------------- > > Key: CASSANDRA-19034 > URL: https://issues.apache.org/jira/browse/CASSANDRA-19034 > Project: Cassandra > Issue Type: Bug > Components: Feature/SAI > Reporter: Branimir Lambov > Assignee: Mike Adamson > Priority: Normal > Fix For: 5.0-beta > > > When run with SAI index, the following two tests error out: > {code} > [junit-timeout] Testcase: > testContainsKeyAndContainsWithIndexOnMapValue(org.apache.cassandra.cql3.validation.operations.SelectTest)-_jdk11: > FAILED > [junit-timeout] Got less rows than expected. Expected 1 but got 0 > [junit-timeout] junit.framework.AssertionFailedError: Got less rows than > expected. Expected 1 but got 0 > [junit-timeout] at > org.apache.cassandra.cql3.CQLTester.assertRows(CQLTester.java:1849) > [junit-timeout] at > org.apache.cassandra.cql3.validation.operations.SelectTest.lambda$testContainsKeyAndContainsWithIndexOnMapValue$9(SelectTest.java:625) > [junit-timeout] at > org.apache.cassandra.cql3.CQLTester.beforeAndAfterFlush(CQLTester.java:2238) > [junit-timeout] at > org.apache.cassandra.cql3.validation.operations.SelectTest.testContainsKeyAndContainsWithIndexOnMapValue(SelectTest.java:618) > [junit-timeout] at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > [junit-timeout] at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > [junit-timeout] at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > [junit-timeout] > [junit-timeout] > [junit-timeout] Testcase: > testFilterWithIndexForContains(org.apache.cassandra.cql3.validation.operations.SelectTest)-_jdk11: > FAILED > [junit-timeout] Invalid value for row 1 column 0 (k1 of type int), expected > <1> but got <0> > [junit-timeout] Invalid value for row 1 column 2 (v of type set<int>), > expected <{4, 5, 6}> but got <{2, 3, 4}> > [junit-timeout] > [junit-timeout] junit.framework.AssertionFailedError: Invalid value for row 1 > column 0 (k1 of type int), expected <1> but got <0> > [junit-timeout] Invalid value for row 1 column 2 (v of type set<int>), > expected <{4, 5, 6}> but got <{2, 3, 4}> > [junit-timeout] > [junit-timeout] at > org.apache.cassandra.cql3.CQLTester.assertRows(CQLTester.java:1826) > [junit-timeout] at > org.apache.cassandra.cql3.validation.operations.SelectTest.lambda$testFilterWithIndexForContains$6(SelectTest.java:543) > [junit-timeout] at > org.apache.cassandra.cql3.CQLTester.beforeAndAfterFlush(CQLTester.java:2240) > [junit-timeout] at > org.apache.cassandra.cql3.validation.operations.SelectTest.testFilterWithIndexForContains(SelectTest.java:542) > [junit-timeout] at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) > [junit-timeout] at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) > [junit-timeout] at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > {code} > The latter seems to be giving the results in the wrong order, and the order > flips when the data is flushed. > Caught during preparation of _latest config that would switch default to SAI > (CASSANDRA-18753). -- This message was sent by Atlassian Jira (v8.20.10#820010) --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org