Github user xenji commented on the pull request: https://github.com/apache/incubator-tinkerpop/pull/246#issuecomment-191424050 Ok, I'll do the rebase asap. I got two questions regarding your implementation advice. The first: When I got you right, you want me to remove the parameter from the "old" constructor and create a new one with the GraphManager as parameter, right? What I do not get is: When I create a new constructor with the GraphManager as parameter, leaving the old one as it is, how should I influence the "bootstrap" process given in the old constructor? Setting the member before calling the other constructor does not work by design and duplicating all the code seems no good option to me. Can you point me in the right direction? The second: The intention behind the PR was - at least from my point of view - exactly what you described. I wanted the original signature of the constructor behave the same way as before. What did I do wrong to not achieve that goal?
--- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---