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.
---

Reply via email to