{"action":"created","issue":{"url":"https://api.github.com/repos/apache/flink/issues/9690","repository_url":"https://api.github.com/repos/apache/flink","labels_url":"https://api.github.com/repos/apache/flink/issues/9690/labels{/name}","comments_url":"https://api.github.com/repos/apache/flink/issues/9690/comments","events_url":"https://api.github.com/repos/apache/flink/issues/9690/events","html_url":"https://github.com/apache/flink/pull/9690","id":494000379,"node_id":"MDExOlB1bGxSZXF1ZXN0MzE3ODM5OTkx","number":9690,"title":"[FLINK-14067] Decouple PlanExecutor from JSON plan generation","user":{"login":"aljoscha","id":68551,"node_id":"MDQ6VXNlcjY4NTUx","avatar_url":"https://avatars1.githubusercontent.com/u/68551?v=4","gravatar_id":"","url":"https://api.github.com/users/aljoscha","html_url":"https://github.com/aljoscha","followers_url":"https://api.github.com/users/aljoscha/followers","following_url":"https://api.github.com/users/aljoscha/following{/other_user}","gists_url":"https://api.github.com/users/aljoscha/gists{/gist_id}","starred_url":"https://api.github.com/users/aljoscha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aljoscha/subscriptions","organizations_url":"https://api.github.com/users/aljoscha/orgs","repos_url":"https://api.github.com/users/aljoscha/repos","events_url":"https://api.github.com/users/aljoscha/events{/privacy}","received_events_url":"https://api.github.com/users/aljoscha/received_events","type":"User","site_admin":false},"labels":[{"id":1273024685,"node_id":"MDU6TGFiZWwxMjczMDI0Njg1","url":"https://api.github.com/repos/apache/flink/labels/component=API/DataSet","name":"component=API/DataSet","color":"175fb7","default":false},{"id":1536123478,"node_id":"MDU6TGFiZWwxNTM2MTIzNDc4","url":"https://api.github.com/repos/apache/flink/labels/component=Client/JobSubmission","name":"component=Client/JobSubmission","color":"175fb7","default":false},{"id":1244454690,"node_id":"MDU6TGFiZWwxMjQ0NDU0Njkw","url":"https://api.github.com/repos/apache/flink/labels/review=description?","name":"review=description?","color":"bcf5db","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2019-09-16T11:47:27Z","updated_at":"2019-09-18T07:54:19Z","closed_at":"2019-09-18T07:54:10Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/flink/pulls/9690","html_url":"https://github.com/apache/flink/pull/9690","diff_url":"https://github.com/apache/flink/pull/9690.diff","patch_url":"https://github.com/apache/flink/pull/9690.patch"},"body":"## What is the purpose of the change\r\n\r\n{{PlanExecutor}} has a method {{getOptimizerPlanAsJSON()}} that is used by DataSet environments to get a JSON version of the execution plan. To ease future work and to make it more maintainable we should get rid of that method and instead have a dedicated utility for generating JSON plans that the environments can use.\r\n\r\n(The only reason this method is on the executor is because only {{flink-clients}} via {{flink-optimizer}} has the required components to derive a JSON plan.)\r\n\r\n\r\n## Brief change log\r\nPlease look at the list of commits for a changelog, each commit is an isolated refactoring on the path to the final result.\r\n\r\n## Verifying this change\r\n\r\n* Added a test for the new `ExecutionPlanUtil`\r\n\r\n## Documentation\r\n\r\n  - Does this pull request introduce a new feature? no\r\n"},"comment":{"url":"https://api.github.com/repos/apache/flink/issues/comments/532566539","html_url":"https://github.com/apache/flink/pull/9690#issuecomment-532566539","issue_url":"https://api.github.com/repos/apache/flink/issues/9690","id":532566539,"node_id":"MDEyOklzc3VlQ29tbWVudDUzMjU2NjUzOQ==","user":{"login":"aljoscha","id":68551,"node_id":"MDQ6VXNlcjY4NTUx","avatar_url":"https://avatars1.githubusercontent.com/u/68551?v=4","gravatar_id":"","url":"https://api.github.com/users/aljoscha","html_url":"https://github.com/aljoscha","followers_url":"https://api.github.com/users/aljoscha/followers","following_url":"https://api.github.com/users/aljoscha/following{/other_user}","gists_url":"https://api.github.com/users/aljoscha/gists{/gist_id}","starred_url":"https://api.github.com/users/aljoscha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aljoscha/subscriptions","organizations_url":"https://api.github.com/users/aljoscha/orgs","repos_url":"https://api.github.com/users/aljoscha/repos","events_url":"https://api.github.com/users/aljoscha/events{/privacy}","received_events_url":"https://api.github.com/users/aljoscha/received_events","type":"User","site_admin":false},"created_at":"2019-09-18T07:54:19Z","updated_at":"2019-09-18T07:54:19Z","author_association":"CONTRIBUTOR","body":"merged"}}
{"actor":{"display_login":"aljoscha"}}