Skip to content

Conversation

@lukasz-antoniak
Copy link
Member

SASI indexes have been removed in DSE 6.9.0.
Error message: Column 'text' has a SASI index but SASI is not supported anymore. This index should removed and probably replaced by an equivalent Storage Attached Index (SAI)

@absurdfarce
Copy link
Contributor

For sake of clarity: this PR makes changes necessary to get the test suite to pass on DSE 6.9. The actual Jenkinsfile changes to enable running tests against 6.9.x are in #1234.

self.assertEqual(TestQueryUpdateModel.objects.count(), 0)

@greaterthanorequalcass3_10
@lessthandse69
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, why does this test... fail on DSE 6.9? Did we remove support for LIKE or something?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, it's because of the removal of SASI.. I see.

Can we replace this with a new SAI test then?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately not the LIKE operator.

SAI does not supports the following query operators for tables with SAI indexes:

  • Strings or Numerics: LIKE, OR, IN

Documentation link.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CEP has it as a "Phase 2" thing which puts it in the same category as OR queries which were discussed at some length for the Java driver (see CASSANDRA-19931). Excluding for 6.9.0 and up seems like the right answer until we get a better answer from SAI.

@absurdfarce absurdfarce changed the title Run integration tests with DSE 6.9.0 PYTHON-1402 Run integration tests with DSE 6.9.0 Oct 15, 2024
@absurdfarce absurdfarce merged commit d15412f into apache:master Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants