{"action":"created","issue":{"url":"https://api.github.com/repos/twitter/heron/issues/1728","repository_url":"https://api.github.com/repos/twitter/heron","labels_url":"https://api.github.com/repos/twitter/heron/issues/1728/labels{/name}","comments_url":"https://api.github.com/repos/twitter/heron/issues/1728/comments","events_url":"https://api.github.com/repos/twitter/heron/issues/1728/events","html_url":"https://github.com/twitter/heron/pull/1728","id":208809482,"number":1728,"title":"upgrade yarn scheduler driver memory to ByteAmount","user":{"login":"mycFelix","id":4719414,"avatar_url":"https://avatars1.githubusercontent.com/u/4719414?v=3","gravatar_id":"","url":"https://api.github.com/users/mycFelix","html_url":"https://github.com/mycFelix","followers_url":"https://api.github.com/users/mycFelix/followers","following_url":"https://api.github.com/users/mycFelix/following{/other_user}","gists_url":"https://api.github.com/users/mycFelix/gists{/gist_id}","starred_url":"https://api.github.com/users/mycFelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mycFelix/subscriptions","organizations_url":"https://api.github.com/users/mycFelix/orgs","repos_url":"https://api.github.com/users/mycFelix/repos","events_url":"https://api.github.com/users/mycFelix/events{/privacy}","received_events_url":"https://api.github.com/users/mycFelix/received_events","type":"User","site_admin":false},"labels":[{"id":265202767,"url":"https://api.github.com/repos/twitter/heron/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":{"login":"mycFelix","id":4719414,"avatar_url":"https://avatars1.githubusercontent.com/u/4719414?v=3","gravatar_id":"","url":"https://api.github.com/users/mycFelix","html_url":"https://github.com/mycFelix","followers_url":"https://api.github.com/users/mycFelix/followers","following_url":"https://api.github.com/users/mycFelix/following{/other_user}","gists_url":"https://api.github.com/users/mycFelix/gists{/gist_id}","starred_url":"https://api.github.com/users/mycFelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mycFelix/subscriptions","organizations_url":"https://api.github.com/users/mycFelix/orgs","repos_url":"https://api.github.com/users/mycFelix/repos","events_url":"https://api.github.com/users/mycFelix/events{/privacy}","received_events_url":"https://api.github.com/users/mycFelix/received_events","type":"User","site_admin":false},"assignees":[{"login":"mycFelix","id":4719414,"avatar_url":"https://avatars1.githubusercontent.com/u/4719414?v=3","gravatar_id":"","url":"https://api.github.com/users/mycFelix","html_url":"https://github.com/mycFelix","followers_url":"https://api.github.com/users/mycFelix/followers","following_url":"https://api.github.com/users/mycFelix/following{/other_user}","gists_url":"https://api.github.com/users/mycFelix/gists{/gist_id}","starred_url":"https://api.github.com/users/mycFelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mycFelix/subscriptions","organizations_url":"https://api.github.com/users/mycFelix/orgs","repos_url":"https://api.github.com/users/mycFelix/repos","events_url":"https://api.github.com/users/mycFelix/events{/privacy}","received_events_url":"https://api.github.com/users/mycFelix/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2017-02-20T08:10:13Z","updated_at":"2017-03-07T09:09:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/twitter/heron/pulls/1728","html_url":"https://github.com/twitter/heron/pull/1728","diff_url":"https://github.com/twitter/heron/pull/1728.diff","patch_url":"https://github.com/twitter/heron/pull/1728.patch"},"body":"As #1727 mentioned and #1707 refactored, this PR upgraded `driverMemory` to use ByteAmount.\r\n\r\nThe key changes are following:\r\n\r\n* add getByteAmountValueMB() in Config class\r\n* change `driverMemory` from int to ByteAmount\r\n* refactor YarnKey class\r\n* remove unused variables in `HeronMasterDriver`\r\n\r\ncc: @billonahill @ashvina "},"comment":{"url":"https://api.github.com/repos/twitter/heron/issues/comments/284663372","html_url":"https://github.com/twitter/heron/pull/1728#issuecomment-284663372","issue_url":"https://api.github.com/repos/twitter/heron/issues/1728","id":284663372,"user":{"login":"mycFelix","id":4719414,"avatar_url":"https://avatars1.githubusercontent.com/u/4719414?v=3","gravatar_id":"","url":"https://api.github.com/users/mycFelix","html_url":"https://github.com/mycFelix","followers_url":"https://api.github.com/users/mycFelix/followers","following_url":"https://api.github.com/users/mycFelix/following{/other_user}","gists_url":"https://api.github.com/users/mycFelix/gists{/gist_id}","starred_url":"https://api.github.com/users/mycFelix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mycFelix/subscriptions","organizations_url":"https://api.github.com/users/mycFelix/orgs","repos_url":"https://api.github.com/users/mycFelix/repos","events_url":"https://api.github.com/users/mycFelix/events{/privacy}","received_events_url":"https://api.github.com/users/mycFelix/received_events","type":"User","site_admin":false},"created_at":"2017-03-07T09:09:21Z","updated_at":"2017-03-07T09:09:21Z","body":"Considering @billonahill 's comment,  creating `ByteAmountUnit` enum class might be not a  proper way.\r\n\r\n@ashvina @billonahill - How about we make [2048](https://github.com/twitter/heron/pull/1728/files#diff-7a5b4ac4a4ebcf917aaa8c34416e02f5R27) into 2048 * 1024 here？"}}
{"actor":{"display_login":"mycFelix"}}