Skip to content

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

Open
rwsegura wants to merge 2 commits into
northwesternmutual:masterfrom
betty-labs:master
Open

Update edge and vertex ID to work with AWS Neptune and Gremlin defaults#40
rwsegura wants to merge 2 commits into
northwesternmutual:masterfrom
betty-labs:master

Conversation

@rwsegura

@rwsegura rwsegura commented Jan 8, 2021

Copy link
Copy Markdown

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

jhole89 commented Feb 12, 2021

Copy link
Copy Markdown

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

@rogaha

rogaha commented Feb 26, 2021

Copy link
Copy Markdown

when is it expected to be merged in ?

@milosgajdos

Copy link
Copy Markdown

Would love this to be merged in.

@damienstamates

Copy link
Copy Markdown
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
Copy Markdown
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

jhole89 commented Mar 18, 2021

Copy link
Copy Markdown

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
Copy Markdown
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