{"action":"created","issue":{"url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/168","labels_url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/168/labels{/name}","comments_url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/168/comments","events_url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/168/events","html_url":"https://github.com/azkaban/azkaban-plugins/pull/168","id":53695199,"number":168,"title":"Ensure that mapred, yarn and hdfs site configuration files are added bef...","user":{"login":"convexquad","id":7005765,"avatar_url":"https://avatars.githubusercontent.com/u/7005765?v=3","gravatar_id":"","url":"https://api.github.com/users/convexquad","html_url":"https://github.com/convexquad","followers_url":"https://api.github.com/users/convexquad/followers","following_url":"https://api.github.com/users/convexquad/following{/other_user}","gists_url":"https://api.github.com/users/convexquad/gists{/gist_id}","starred_url":"https://api.github.com/users/convexquad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/convexquad/subscriptions","organizations_url":"https://api.github.com/users/convexquad/orgs","repos_url":"https://api.github.com/users/convexquad/repos","events_url":"https://api.github.com/users/convexquad/events{/privacy}","received_events_url":"https://api.github.com/users/convexquad/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-01-07T23:16:10Z","updated_at":"2015-01-26T01:32:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/azkaban/azkaban-plugins/pulls/168","html_url":"https://github.com/azkaban/azkaban-plugins/pull/168","diff_url":"https://github.com/azkaban/azkaban-plugins/pull/168.diff","patch_url":"https://github.com/azkaban/azkaban-plugins/pull/168.patch"},"body":"...ore the configuration file containing properties to inject.\r\n\r\nCurrently, injected properties are being overridden by properties appearing in the site configs. This pull request will fix this problem. Users will not be able to override properties marked \"final\" in the site configs, which is a good thing.\r\n\r\nI checked all the Hadoop source code to make sure that adding the site configs here is safe to do in both Hadoop 1 and Hadoop 2. Adding them is safe even if you Hadoop installation does not have one of the listed site config files.\r\n\r\nTested successfully on lva1-spadesaz01 (an internal node at LinkedIn)."},"comment":{"url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/comments/71404858","html_url":"https://github.com/azkaban/azkaban-plugins/pull/168#issuecomment-71404858","issue_url":"https://api.github.com/repos/azkaban/azkaban-plugins/issues/168","id":71404858,"user":{"login":"hluu","id":533460,"avatar_url":"https://avatars.githubusercontent.com/u/533460?v=3","gravatar_id":"","url":"https://api.github.com/users/hluu","html_url":"https://github.com/hluu","followers_url":"https://api.github.com/users/hluu/followers","following_url":"https://api.github.com/users/hluu/following{/other_user}","gists_url":"https://api.github.com/users/hluu/gists{/gist_id}","starred_url":"https://api.github.com/users/hluu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hluu/subscriptions","organizations_url":"https://api.github.com/users/hluu/orgs","repos_url":"https://api.github.com/users/hluu/repos","events_url":"https://api.github.com/users/hluu/events{/privacy}","received_events_url":"https://api.github.com/users/hluu/received_events","type":"User","site_admin":false},"created_at":"2015-01-26T01:32:45Z","updated_at":"2015-01-26T01:32:45Z","body":"LGTM"}}