{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-rya/issues/118","repository_url":"https://api.github.com/repos/apache/incubator-rya","labels_url":"https://api.github.com/repos/apache/incubator-rya/issues/118/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-rya/issues/118/comments","events_url":"https://api.github.com/repos/apache/incubator-rya/issues/118/events","html_url":"https://github.com/apache/incubator-rya/pull/118","id":184393051,"number":118,"title":"RYA-174 Removing Mongo Test Factory from non-test code","user":{"login":"amihalik","id":4710489,"avatar_url":"https://avatars.githubusercontent.com/u/4710489?v=3","gravatar_id":"","url":"https://api.github.com/users/amihalik","html_url":"https://github.com/amihalik","followers_url":"https://api.github.com/users/amihalik/followers","following_url":"https://api.github.com/users/amihalik/following{/other_user}","gists_url":"https://api.github.com/users/amihalik/gists{/gist_id}","starred_url":"https://api.github.com/users/amihalik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amihalik/subscriptions","organizations_url":"https://api.github.com/users/amihalik/orgs","repos_url":"https://api.github.com/users/amihalik/repos","events_url":"https://api.github.com/users/amihalik/events{/privacy}","received_events_url":"https://api.github.com/users/amihalik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-10-21T03:43:30Z","updated_at":"2016-10-24T16:59:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-rya/pulls/118","html_url":"https://github.com/apache/incubator-rya/pull/118","diff_url":"https://github.com/apache/incubator-rya/pull/118.diff","patch_url":"https://github.com/apache/incubator-rya/pull/118.patch"},"body":">What Changed?\r\n\r\nRemoved Mongo Test Factory code from non-test code\r\n\r\n### Tests\r\n\r\nno new tests\r\n\r\n### Links\r\n[Jira](https://issues.apache.org/jira/browse/RYA-174)\r\n\r\n### Checklist\r\n- [ ] Code Review\r\n- [ ] Squash Commits\r\n\r\n#### People To Reivew\r\n\r\n@dlotts\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/apache/incubator-rya/issues/comments/255799850","html_url":"https://github.com/apache/incubator-rya/pull/118#issuecomment-255799850","issue_url":"https://api.github.com/repos/apache/incubator-rya/issues/118","id":255799850,"user":{"login":"pujav65","id":6528884,"avatar_url":"https://avatars.githubusercontent.com/u/6528884?v=3","gravatar_id":"","url":"https://api.github.com/users/pujav65","html_url":"https://github.com/pujav65","followers_url":"https://api.github.com/users/pujav65/followers","following_url":"https://api.github.com/users/pujav65/following{/other_user}","gists_url":"https://api.github.com/users/pujav65/gists{/gist_id}","starred_url":"https://api.github.com/users/pujav65/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pujav65/subscriptions","organizations_url":"https://api.github.com/users/pujav65/orgs","repos_url":"https://api.github.com/users/pujav65/repos","events_url":"https://api.github.com/users/pujav65/events{/privacy}","received_events_url":"https://api.github.com/users/pujav65/received_events","type":"User","site_admin":false},"created_at":"2016-10-24T16:59:09Z","updated_at":"2016-10-24T16:59:09Z","body":"hey i see where this was coming from, but i think we should try to move to the mongo client being passed in.  I'd like to see this look a lot closer to what was done in Accumulo (more because the Accumulo implementation is in line with what the intended contracts of the RyaDAO/etc. are).   We started moving away from that when we started having a connections factory -- I think this pull request removes that which is good, but I agree with Aaron that we should move to have the connector/client be passed in to everything that needs it rather than passing in the connection information."}}
{"actor":{"display_login":"pujav65"}}