{"comment":{"url":"https://api.github.com/repos/derekwaynecarr/kubernetes/comments/15867067","html_url":"https://github.com/derekwaynecarr/kubernetes/commit/55a40f01006a0b470cbdacec9ceecbd65f490e4b#commitcomment-15867067","id":15867067,"user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"position":36,"line":191,"path":"contrib/mesos/pkg/controllermanager/controllermanager.go","commit_id":"55a40f01006a0b470cbdacec9ceecbd65f490e4b","created_at":"2016-02-03T19:51:59Z","updated_at":"2016-02-03T19:51:59Z","body":"Maybe a future factoring, but why do we need a Factory if we have options?  Won't options cover the 90% case?  Or do you want a factory to hide the testing logic."}}