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

Update edge and vertex ID to work with AWS Neptune and Gremlin defaults #40

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rwsegura
Copy link

@rwsegura rwsegura commented Jan 8, 2021

Currently users who are using grammes have to switch between v1.1.2 for local and v1.2.0 for AWS Neptune. This will allow users to work with both of them.

@jhole89
Copy link

jhole89 commented Feb 12, 2021

This is great - can't wait for it to get merged. Thanks @rwsegura

@rogaha
Copy link

rogaha commented Feb 26, 2021

when is it expected to be merged in ?

@milosgajdos
Copy link

Would love this to be merged in.

@damienstamates
Copy link
Contributor

This is a great contribution, but unfortunately I cannot accept this. Golang is a prohibited language for Northwestern Mutual and Grammes in turn is low on the priority list. This along with the fact that I am no longer with Northwestern Mutual makes this impossible. I would suggest finding a fork of the project that is updated with something like this.

Thanks

@rwsegura
Copy link
Author

I would be willingly to make a new org that forks the code and maintain the repo. Using graph database is one of our core features at the moment

@jhole89
Copy link

jhole89 commented Mar 18, 2021

Happy to help out with this. If Northwestern Mutual have no interest in maintaining this project then having a community supported org that maintains it would be great.

@rwsegura
Copy link
Author

I have made a new org and forked the repo at https://github.com/grammes-go/grammes. Feel free to redirect PRs there and reach out if you'd like to contribute

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.

6 participants