{"action":"created","issue":{"url":"https://api.github.com/repos/linkedin/gobblin/issues/677","repository_url":"https://api.github.com/repos/linkedin/gobblin","labels_url":"https://api.github.com/repos/linkedin/gobblin/issues/677/labels{/name}","comments_url":"https://api.github.com/repos/linkedin/gobblin/issues/677/comments","events_url":"https://api.github.com/repos/linkedin/gobblin/issues/677/events","html_url":"https://github.com/linkedin/gobblin/pull/677","id":132284568,"number":677,"title":"Fix race condition in FsStateStore","user":{"login":"zliu41","id":6342538,"avatar_url":"https://avatars.githubusercontent.com/u/6342538?v=3","gravatar_id":"","url":"https://api.github.com/users/zliu41","html_url":"https://github.com/zliu41","followers_url":"https://api.github.com/users/zliu41/followers","following_url":"https://api.github.com/users/zliu41/following{/other_user}","gists_url":"https://api.github.com/users/zliu41/gists{/gist_id}","starred_url":"https://api.github.com/users/zliu41/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zliu41/subscriptions","organizations_url":"https://api.github.com/users/zliu41/orgs","repos_url":"https://api.github.com/users/zliu41/repos","events_url":"https://api.github.com/users/zliu41/events{/privacy}","received_events_url":"https://api.github.com/users/zliu41/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2016-02-08T23:17:45Z","updated_at":"2016-02-10T16:49:43Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/linkedin/gobblin/pulls/677","html_url":"https://github.com/linkedin/gobblin/pull/677","diff_url":"https://github.com/linkedin/gobblin/pull/677.diff","patch_url":"https://github.com/linkedin/gobblin/pull/677.patch"},"body":"@ibuenros can you review?"},"comment":{"url":"https://api.github.com/repos/linkedin/gobblin/issues/comments/182475304","html_url":"https://github.com/linkedin/gobblin/pull/677#issuecomment-182475304","issue_url":"https://api.github.com/repos/linkedin/gobblin/issues/677","id":182475304,"user":{"login":"kadaan","id":1904898,"avatar_url":"https://avatars.githubusercontent.com/u/1904898?v=3","gravatar_id":"","url":"https://api.github.com/users/kadaan","html_url":"https://github.com/kadaan","followers_url":"https://api.github.com/users/kadaan/followers","following_url":"https://api.github.com/users/kadaan/following{/other_user}","gists_url":"https://api.github.com/users/kadaan/gists{/gist_id}","starred_url":"https://api.github.com/users/kadaan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kadaan/subscriptions","organizations_url":"https://api.github.com/users/kadaan/orgs","repos_url":"https://api.github.com/users/kadaan/repos","events_url":"https://api.github.com/users/kadaan/events{/privacy}","received_events_url":"https://api.github.com/users/kadaan/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T16:49:43Z","updated_at":"2016-02-10T16:49:43Z","body":"That would be great, but I think users of gobblin should be able to provide new decorators without having to extend gobblin itself.\n\n> On Feb 10, 2016, at 8:48 AM, Issac Buenrostro <notifications@github.com> wrote:\n> \n> I understand your point, but I don't think having our own interface is the correct solution: we would have to manually extend all implementations of hadoop's FileSystem, and it would be harder for users to benefit from already existing implementations.\n> Maybe we could have a FileSystem decorator instead with a few of the methods you mention. We could then detect certain known schemes with incompatibilities and apply fixes across the entire code, but any FileSystem we haven't considered yet would still work appropriately.\n> \n> —\n> Reply to this email directly or view it on GitHub <https://github.com/linkedin/gobblin/pull/677#issuecomment-182474824>.\n> \n\n"}}