[janusgraph-foundationdb] initial fork options

69 views
Skip to first unread message

Jason Plurad

unread,
Jun 16, 2020, 10:14:00 AM6/16/20
to JanusGraph developers
For the new janusgraph-foundationdb repo, we need a starting point. Here are a few options:

1. Fork from experoinc/janusgraph-foundationdb into JanusGraph/janusgraph-foundationdb. The external fork authors submit PRs with their changes to the new repo, and the code changes will be reviewed before commit.

2. Fork from one of the external janusgraph-foundationdb forks into JanusGraph/janusgraph-foundationdb. rngcntr/janusgraph-foundationdb has the most commits among the three. The other two forks would submit PRs to the new repo.

3. Have the 3 authors merge their forks first, then fork the unified fork into JanusGraph/janusgraph-foundationdb. This could provide a unified repo before imposing code review process.

I haven't reviewed any of the code out there, so I don't have a strong opinion on any of these options. Open for more ideas or suggestions from the authors, committers, and others in the community.

-- Jason

Misha Brukman

unread,
Jun 16, 2020, 10:23:27 AM6/16/20
to JanusGraph developer list
I'd like to propose another option:

4. Ask Ted & Expero if they wouldn't mind transferring the initial repo experoinf/janusgraph-foundationdb into the JanusGraph org as-is so it becomes the authoritative home (rather than forking it), and then each of the existing fork owners can propose PRs to merge their changes incrementally into the new base repo.

--
You received this message because you are subscribed to the Google Groups "JanusGraph developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to janusgraph-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/janusgraph-dev/ed630d2d-cf9d-44ab-85d4-27042208e932o%40googlegroups.com.

f.gri...@gmail.com

unread,
Jun 18, 2020, 5:15:57 AM6/18/20
to JanusGraph developers
Of course from my point of view, the easiest option is #2, taking my repo (rngcntr/janusgraph-foundationdb) as the base ;)
But to be honest, I don't think this is the cleanest way to go. I developed most of this alone and without review of others. Furthermore, I refactored much of the existing code. I can't speak for the other authors, but I think the situation might be similar for them.

So my suggestion would be to go for #1 or even better #4 as suggested by Misha.
Message has been deleted

Ted Wilmes

unread,
Jun 19, 2020, 9:43:47 PM6/19/20
to JanusGraph developers
Four makes good sense to me. I don't think that'll be an issue on our end but will confirm.

--Ted

Christopher Jackson

unread,
Jun 20, 2020, 2:42:15 AM6/20/20
to JanusGraph developers
My suggestion would also be either option #1 or option #4.

Jason Plurad

unread,
Jun 24, 2020, 1:40:36 PM6/24/20
to JanusGraph developers
Thanks Ted. Chime back on this thread once you get clearance from your company, then we can move forward after the experoinc repo is transferred over.

Ted Wilmes

unread,
Jun 26, 2020, 3:22:43 PM6/26/20
to JanusGraph developer list
I have approval on our side. I'll let you all know when it's been transferred over.

Thanks,
Ted

--
You received this message because you are subscribed to a topic in the Google Groups "JanusGraph developers" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/janusgraph-dev/tleruwXYBzU/unsubscribe.
To unsubscribe from this group and all its topics, send an email to janusgraph-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/janusgraph-dev/1bd113bc-877a-4479-93d4-4f8fa7752d4bn%40googlegroups.com.

Ted Wilmes

unread,
Jun 30, 2020, 10:54:03 AM6/30/20
to JanusGraph developer list
Success! It is transferred over: https://github.com/JanusGraph/janusgraph-foundationdb. I'm excited to see things progress on the FDB front.

Thanks,
Ted

Misha Brukman

unread,
Jun 30, 2020, 11:29:12 AM6/30/20
to JanusGraph developer list
That's great to hear! Thank you to Ted & Expero for contributing your FoundationDB backend to the Janusgraph Project!

You received this message because you are subscribed to the Google Groups "JanusGraph developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email to janusgraph-de...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/janusgraph-dev/CAGrm2%3DBq-_eo-OveMjkPBTQDF4cS83EJ-uY%3DYn%2BhEaY7tbQ-PA%40mail.gmail.com.

Christopher Jackson

unread,
Jul 4, 2020, 1:05:00 PM7/4/20
to JanusGraph developers
Thanks Ted and Expero for contributing this code and migrating it to its new home so quickly!
Reply all
Reply to author
Forward
0 new messages