Skip to content

[GEOMESA-3435] Added scyllaDB tests and documentation#3437

Merged
elahrvivaz merged 4 commits intolocationtech:mainfrom
andrii-kysylevskyi:GEOMESA-3435-add-scylladb-to-docs-2
Oct 28, 2025
Merged

[GEOMESA-3435] Added scyllaDB tests and documentation#3437
elahrvivaz merged 4 commits intolocationtech:mainfrom
andrii-kysylevskyi:GEOMESA-3435-add-scylladb-to-docs-2

Conversation

@andrii-kysylevskyi
Copy link
Contributor

This PR is based on this original PR.

As stated in the original:

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.

This PR intends to achieve the same goal, but is aligned with the latest state of the Geomesa project and applies the approach suggested by @elahrvivaz

@andrii-kysylevskyi andrii-kysylevskyi force-pushed the GEOMESA-3435-add-scylladb-to-docs-2 branch from cd5e781 to 1b6a23f Compare October 27, 2025 09:17
@elahrvivaz
Copy link
Contributor

Thanks for picking up this work!

…versions, applied PR review comments. Used the latest open source version
@andrii-kysylevskyi
Copy link
Contributor Author

andrii-kysylevskyi commented Oct 27, 2025

Thanks for picking up this work!

@elahrvivaz Thanks a lot for the review! Applied all the changes requested, compiled the docs locally - worked!
I also used the latest version of the open source ScyllaDB variant instead of the Enterprise one used before by mistake. Tests ran fine with it as well, so thought that made more sense

@elahrvivaz elahrvivaz merged commit 59bf958 into locationtech:main Oct 28, 2025
80 checks passed
@elahrvivaz
Copy link
Contributor

thanks for the quick responses!

@andrii-kysylevskyi
Copy link
Contributor Author

thanks for the quick responses!

Thank you as well, @elahrvivaz!
Can I confirm if the docs should have already been updated, or need some time to deploy/get updated?

@elahrvivaz
Copy link
Contributor

thanks for the quick responses!

Thank you as well, @elahrvivaz! Can I confirm if the docs should have already been updated, or need some time to deploy/get updated?

the docs run on a nightly schedule, so they should be updated around 5AM EST tomorrow.

@andrii-kysylevskyi
Copy link
Contributor Author

@elahrvivaz Hi Emilio, I have just double-checked the docs and could not find the new references to Scylla there. Is there any chance the docs pipeline did not pass, or those new references were omitted somehow? Thanks a lot in advance

@elahrvivaz
Copy link
Contributor

@elahrvivaz Hi Emilio, I have just double-checked the docs and could not find the new references to Scylla there. Is there any chance the docs pipeline did not pass, or those new references were omitted somehow? Thanks a lot in advance

It's right here: https://www.geomesa.org/documentation/latest/user/scylladb/index.html

The nightly docs go to 'latest', while the current release goes to 'stable'. They're both linked off the home page though.

Thanks,

@andrii-kysylevskyi
Copy link
Contributor Author

@elahrvivaz Hi Emilio, I have just double-checked the docs and could not find the new references to Scylla there. Is there any chance the docs pipeline did not pass, or those new references were omitted somehow? Thanks a lot in advance

It's right here: https://www.geomesa.org/documentation/latest/user/scylladb/index.html

The nightly docs go to 'latest', while the current release goes to 'stable'. They're both linked off the home page though.

Thanks,

Aah, thank you so much, makes sense. After looking through the docs, I have noticed that I have completely missed the quick-start section. Have just submitted another PR here.

I mentioned in the description there one problem that I suspect might be happening, but might be worth switching 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.

2 participants