northwesternmutual / grammes

A Go package built to communicate with Apache TinkerPop™ Graph computing framework using Gremlin; a graph traversal language used by graph databases such as JanusGraph®, MS Cosmos DB, AWS Neptune, and DataStax® Enterprise Graph.
Apache License 2.0
125 stars 45 forks source link

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

Open rwsegura opened 3 years ago

rwsegura commented 3 years ago

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 commented 3 years ago

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

rogaha commented 3 years ago

when is it expected to be merged in ?

milosgajdos commented 3 years ago

Would love this to be merged in.

damienstamates commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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