{"action":"created","issue":{"url":"https://api.github.com/repos/apache/incubator-reef/issues/164","labels_url":"https://api.github.com/repos/apache/incubator-reef/issues/164/labels{/name}","comments_url":"https://api.github.com/repos/apache/incubator-reef/issues/164/comments","events_url":"https://api.github.com/repos/apache/incubator-reef/issues/164/events","html_url":"https://github.com/apache/incubator-reef/pull/164","id":71398400,"number":164,"title":"[REEF-283] Introduce RemoteManagerFactory","user":{"login":"markusweimer","id":502371,"avatar_url":"https://avatars.githubusercontent.com/u/502371?v=3","gravatar_id":"","url":"https://api.github.com/users/markusweimer","html_url":"https://github.com/markusweimer","followers_url":"https://api.github.com/users/markusweimer/followers","following_url":"https://api.github.com/users/markusweimer/following{/other_user}","gists_url":"https://api.github.com/users/markusweimer/gists{/gist_id}","starred_url":"https://api.github.com/users/markusweimer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markusweimer/subscriptions","organizations_url":"https://api.github.com/users/markusweimer/orgs","repos_url":"https://api.github.com/users/markusweimer/repos","events_url":"https://api.github.com/users/markusweimer/events{/privacy}","received_events_url":"https://api.github.com/users/markusweimer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-04-27T20:50:32Z","updated_at":"2015-04-28T02:08:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/incubator-reef/pulls/164","html_url":"https://github.com/apache/incubator-reef/pull/164","diff_url":"https://github.com/apache/incubator-reef/pull/164.diff","patch_url":"https://github.com/apache/incubator-reef/pull/164.patch"},"body":"This change adds the new interface `RemoteManagerFactory` together with its default implementation `DefaultRemoteManagerFactory`. The factory is injectable, yet offers all the constructors of `DefaultRemoteManager` through its methods. This allows us to mix and match between Tang Configuration and ad-hoc overrides.\r\n\r\nJIRA:\r\n  [REEF-283](https://issues.apache.org/jira/browse/REEF-283)"},"comment":{"url":"https://api.github.com/repos/apache/incubator-reef/issues/comments/96876264","html_url":"https://github.com/apache/incubator-reef/pull/164#issuecomment-96876264","issue_url":"https://api.github.com/repos/apache/incubator-reef/issues/164","id":96876264,"user":{"login":"dafrista","id":1514239,"avatar_url":"https://avatars.githubusercontent.com/u/1514239?v=3","gravatar_id":"","url":"https://api.github.com/users/dafrista","html_url":"https://github.com/dafrista","followers_url":"https://api.github.com/users/dafrista/followers","following_url":"https://api.github.com/users/dafrista/following{/other_user}","gists_url":"https://api.github.com/users/dafrista/gists{/gist_id}","starred_url":"https://api.github.com/users/dafrista/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dafrista/subscriptions","organizations_url":"https://api.github.com/users/dafrista/orgs","repos_url":"https://api.github.com/users/dafrista/repos","events_url":"https://api.github.com/users/dafrista/events{/privacy}","received_events_url":"https://api.github.com/users/dafrista/received_events","type":"User","site_admin":false},"created_at":"2015-04-28T02:08:56Z","updated_at":"2015-04-28T02:08:56Z","body":"@bgchun These constructors are reflected in the newly introduced Factory interface as well. It might make sense to first clean these up before introducing the new interface (and possibly having to change the interfaces as well). @markusweimer since you've made the PR, I think it's fine if you decide :)"}}