search

datastax / adelphi

Automation tool for testing C* OSS that assembles cassandra-diff, nosqlbench, fqltool
Apache License 2.0
10 stars 6 forks source link

Fix testing support for SASI indexes #105

Open absurdfarce opened 3 years ago

absurdfarce commented 3 years ago

Bug Report

Description

Current integration test setup adds SASI indexes to schema created in Docker C instance (see, for example, https://github.com/datastax/adelphi/blob/817aff9a08ba323623577cbfc98c62a29c8b13f3/python/adelphi/integration-tests/base-schema.cql#L47). When we spin up a Docker instance this operation fails for C 4.0-beta3 since SASI must be explicitly enabled in cassandra.yaml before these indexes can be created. We then run "adelphi" against this schema and compare the exported CQL to a reference schema CQL. The reference schema does not include the SASI indexes so the test passes, although SASI was never really tested in this scheme.

Similar issues were noted (for very similar reasons) with the Python impl being worked on for https://github.com/datastax/adelphi/issues/62.

We should explore making SASI indexes a supported feature of this testing regime. The obvious answer is to explicitly support SASI in the configs (at least for C* 4.0).

┆Issue is synchronized with this Jira Task by Unito ┆Issue Number: AD-41

absurdfarce commented 3 years ago

The Docker container in question has some support for running with a modified cassandra.yaml but it isn't... a great story (see https://hub.docker.com/_/cassandra).