janusgraph icon indicating copy to clipboard operation
janusgraph copied to clipboard

Officially support Scylla

Open FlorianHockmann opened this issue 4 years ago • 6 comments

We basically already support running JanusGraph with Scylla but we don't communicate that anywhere. I suggest that we add Scylla to our test suite running with GitHub actions and then add it to the list of tested projects in our changelog and in the release notes. Afterwards we could add a small section to the docs explaining how to use Scylla with the CQL backend.

  • [x] Test Scylla automatically with GitHub actions #2351
  • [x] Fix Hadoop integration tests for Scylla #2505
  • [x] Document tested versions in changelog and release notes
  • [x] Document how to use Scylla with the CQL backend (fixed in #3058)
  • [ ] Fix SSL integration tests for Scylla #3595
  • [ ] Add possibility to switch between scylla and cql dependencies in JanusGraph Server #3580

FlorianHockmann avatar Feb 12 '21 08:02 FlorianHockmann

Also, it would be good to add this feature (which is partially related): https://github.com/JanusGraph/janusgraph/issues/1778

porunov avatar Feb 12 '21 08:02 porunov

There is already a PR https://github.com/JanusGraph/janusgraph/pull/2352 for testing.

farodin91 avatar Feb 12 '21 08:02 farodin91

I think #1778 is going a step further already. I would like to keep this issue quite simple so that we basically just document how Scylla can already be used with JanusGraph.

There is already a PR #2352 for testing.

Nice to see! Then we really only have to change the docs here.

FlorianHockmann avatar Feb 12 '21 08:02 FlorianHockmann

https://github.com/JanusGraph/janusgraph/pull/2352 is merged with https://github.com/JanusGraph/janusgraph/issues/2505 left as TODO.

li-boxuan avatar Mar 07 '21 08:03 li-boxuan

#2352 is merged with #2505 left as TODO.

Thanks, I added #2505 as a TODO to the description.

FlorianHockmann avatar May 28 '21 09:05 FlorianHockmann

Just realized that the docs part is also already done: #3058

FlorianHockmann avatar Jul 08 '22 15:07 FlorianHockmann