-
Notifications
You must be signed in to change notification settings - Fork 441
[GEOMESA-3435] Added scyllaDB quick-start guide and README pointer #3452
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
base: main
Are you sure you want to change the base?
[GEOMESA-3435] Added scyllaDB quick-start guide and README pointer #3452
Conversation
|
hmm good question - we can't easily go back and update 'stable' since that's built off a tag. we could back-port it for the next bug-fix release, but I don't have any planned imminently. we could point it at 'latest', but it's not going to be a smooth experience for less technical users (since there are references to releases and such that don't exist yet). or we could hold off on merging the README changes until closer to the release. what would be your preference? |
|
I might also prefer to change the individual quickstart links to a single link pointing to https://www.geomesa.org/documentation/stable/tutorials/index.html#quick-starts so we don't have to keep them in sync. that would solve the issue of merging this now, although the tutorial would still not be available on stable. |
Great point on the quick-starts, it does feel like a better way for sure. Happy to do it here if that's okay, or can open a separate PR.
Regarding the stable vs latest, on one hand, ScyllaDB seems to be supported by the stable version, so it would make sense to tell users that. Also, I am still struggling to find the Would you consider mentioning the latest documentation in a README file, maybe? (unless I missed it and it's already there) |
| [Accumulo](https://www.geomesa.org/documentation/tutorials/geomesa-quickstart-accumulo.html) | | ||
| [HBase](https://www.geomesa.org/documentation/tutorials/geomesa-quickstart-hbase.html) | | ||
| [Cassandra](https://www.geomesa.org/documentation/tutorials/geomesa-quickstart-cassandra.html) | | ||
| [ScyllaDB](https://www.geomesa.org/documentation/tutorials/geomesa-quickstart-scylladb.html) | |
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.
<b>[Documentation](https://www.geomesa.org/documentation/) | [Upgrade Guide](https://www.geomesa.org/documentation/user/upgrade.html) | [Quick Starts](https://www.geomesa.org/documentation/stable/tutorials/index.html#quick-starts) | [Tutorials](https://www.geomesa.org/tutorials/) | [Nightly snapshots](https://geomesa.org/documentation/latest/)</b>


This is an addition of the documentation aspects missed in the previous PR here
I am not sure about one thing @elahrvivaz has mentioned about stable vs latest versions of the docs.
The URLs in README seem to point to /documentation, which is redirected to /stable version.
While Scylla docs might not be in the
stableversion as it is getting deployed to /latest instead.Emilio, do you think this could cause issues and, if yes, then how would you recommend to proceed? Somehow deploy to stable or point the URL to the
latestinstead? Thanks a lot in advance!