-
Notifications
You must be signed in to change notification settings - Fork 438
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
base: main
Are you sure you want to change the base?
GEOMESA-3435. mention scylladb in docs #3255
Conversation
docs/user/cassandra/install.rst
Outdated
configuration file (located at ``conf/cassandra.yaml`` in your Cassandra | ||
installation directory). | ||
installation directory, ``/etc/scylla/scylla.yaml`` for ScyllaDB accordingly). |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
Thanks for adding the docs! I don't think we should replace all mentions of Cassandra with Cassandra/ScyllaDB - how about adding a new |
…dd scylladb support to docs
392150f
to
6675d5a
Compare
changes look reasonable to me, but the build is failing. @jnh5y do you have any comments? |
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.