Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GEOMESA-3435. mention scylladb in docs #3255

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mkorolyov
Copy link

ScyllaDB is a drop in more performant replacement for Cassandra. It works out of the box with GeoMesa just by replacing connection string from Cassandra to ScyllaDB. Lets mention this in docs so people will be aware of such option.

configuration file (located at ``conf/cassandra.yaml`` in your Cassandra
installation directory).
installation directory, ``/etc/scylla/scylla.yaml`` for ScyllaDB accordingly).
Copy link
Contributor

Choose a reason for hiding this comment

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

Have you tested ScyllaDB with GeoMesa?

Is there a way that GeoMesa committers can test with ScyllaDB?

Copy link
Author

Choose a reason for hiding this comment

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

Hey @jnh5y ,

Yes, I've tested ScyllaDB with GeoMesa. Cassandra quick start example works, all you need to do is to have running ScyllaDB on 127.0.0.1:9042(same default port) instead of Cassandra.

Regarding tests: original Cassandra module tests are using an embedded Cassandra server. This could be migrated to testcontainers, so we would run the same tests both for Cassandra and ScyllaDB. WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

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

switching the cassandra module to use testcontainers would be nice - we just haven't gotten around to doing so.

@elahrvivaz
Copy link
Contributor

Thanks for adding the docs! I don't think we should replace all mentions of Cassandra with Cassandra/ScyllaDB - how about adding a new scylladb/index.rst file that comes after Cassandra and just says that the Cassandra store is also compatible with ScyllaDB? And if there are any differences (e.g. /etc/scylla/scylla.yaml for config), call them out there.

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.

3 participants