Github user pujav65 commented on the issue:
https://github.com/apache/incubator-rya/pull/118
i thought the mongo connections factory was a bad idea at the time. so i
think removing it is a good thing in general. The instance driver you're
referring to is in the merge tool -- its not used in the main client path. The
accumulo dao has a setConnector() method which really should have been emulated
by the Mongo dao. I didn't realize the pattern until we started having issues
with setting up multiple connections, but the passing in the connector/client
pattern is nice because its a lot more flexible than what we currently have.
Regardless, I think this pull request is fine -- all of this discussion is on
future direction.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
