Thanks for the write up David - note that this is similar to what I wrote a few days back on this thread:
https://lists.apache.org/thread.html/d16575fc2fa2784f714cf18bb7a371e7a7bba0e3eba06cc80ec3a3de@%3Cdev.tinkerpop.apache.org%3E There have been no objections to this direction so lazy consensus has been achieved at this point. On Wed, Mar 15, 2017 at 4:09 PM, David Pitera <[email protected]> wrote: > Hey all, > > There has been a good amount of discussion on the issue: > https://issues.apache.org/jira/browse/TINKERPOP-1438 and on the > corresponding PR: https://github.com/apache/tinkerpop/pull/569 > > This change is considered `breaking` because if any users were currently > using the existing final class GraphManager, then this change would be > breaking. Note that the GraphManager class was intended to be used > internally, so as per some discussion already on the PR, this is one of > those OK breaking changes: https://github.com/apache/tinkerpop/pull/569# > issuecomment-285705316. > > I have updated all relevant documentation links to include information on > how to fix the change should the need arise, and how to make use of the > functionality supplied by my PR. > > I have updated my PR with all suggestions/comments, so it should be ready > for another review. > > Just sending out this email to ensure the larger community is aware that > there is technically a breaking change that I want to merge into tp32 and > then master. > > Cheers, > David >
