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

Add DB adaptor for RedisGraph #84

Open
onyame opened this issue Dec 16, 2019 · 6 comments
Open

Add DB adaptor for RedisGraph #84

onyame opened this issue Dec 16, 2019 · 6 comments
Assignees

Comments

@onyame
Copy link
Member

onyame commented Dec 16, 2019

Adaptor to store PROV in RedisGraph (https://oss.redislabs.com/redisgraph/)

@B-Stefan
Copy link
Collaborator

Nice I will look into that if I have some time during the holidays! 👍

@B-Stefan
Copy link
Collaborator

Thanks, Andreas, for the hint with the module. I think this could be a great new feature for the lib.

I evaluated the current status of the Redis graph project and created a minimal implementation of the Redis adapter. The technology is promising but in an early stage of development. I want to share some thoughts I had during the implementation :

  • No stable docker image redislabs/redisgraph:edge needed to get it running / :latest is 4 months old
  • No stable python lib - I had to install it directly via git git+git://github.com/RedisGraph/redisgraph-py
  • Some Cypher features missing - see blocking issues

Todos

  • Proper session support (transactions?)
  • Redis authentication support (username / password)
  • Redis host / port via config instead of static values
  • Support merge feature (Curretly not possible)

Blocking issues

Currently, we need to rewrite the Chyper queries and rethink the logic in the adapter itself. That will be a lot of work if you like to support the full feature set. The other "solution" would be to wait a couple of months and hope that the Redis team will implement more Chyper features and solve some blocking issues.

I don't have the time to rewrite the whole adapter code, but If someone likes to deep dive into this, I'm happy to support it!
Otherwise, I will look into this again and provide an update In a couple of months.

@B-Stefan
Copy link
Collaborator

@B-Stefan B-Stefan self-assigned this Jan 26, 2020
@B-Stefan
Copy link
Collaborator

There seems to be little activity in the RedisGraph repository. I don't think that RedisGraph will support the full feature set of cypher in the near future.

To implement this feature a valid option is to rewrite and rethink/optimize the queries itself. By using only the already provided features within RedisGraph

@B-Stefan
Copy link
Collaborator

B-Stefan commented Dec 6, 2020

Update: The issue is still open but I requested an update.

1 similar comment
@B-Stefan
Copy link
Collaborator

B-Stefan commented Jan 6, 2022

Update: The issue is still open but I requested an update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants