{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto/issues/11910","repository_url":"https://api.github.com/repos/prestodb/presto","labels_url":"https://api.github.com/repos/prestodb/presto/issues/11910/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto/issues/11910/comments","events_url":"https://api.github.com/repos/prestodb/presto/issues/11910/events","html_url":"https://github.com/prestodb/presto/pull/11910","id":380085170,"node_id":"MDExOlB1bGxSZXF1ZXN0MjMwMzY3MTE0","number":11910,"title":"Add spilled data in query summary, Web UI","user":{"login":"anoopj","id":269864,"node_id":"MDQ6VXNlcjI2OTg2NA==","avatar_url":"https://avatars1.githubusercontent.com/u/269864?v=4","gravatar_id":"","url":"https://api.github.com/users/anoopj","html_url":"https://github.com/anoopj","followers_url":"https://api.github.com/users/anoopj/followers","following_url":"https://api.github.com/users/anoopj/following{/other_user}","gists_url":"https://api.github.com/users/anoopj/gists{/gist_id}","starred_url":"https://api.github.com/users/anoopj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anoopj/subscriptions","organizations_url":"https://api.github.com/users/anoopj/orgs","repos_url":"https://api.github.com/users/anoopj/repos","events_url":"https://api.github.com/users/anoopj/events{/privacy}","received_events_url":"https://api.github.com/users/anoopj/received_events","type":"User","site_admin":false},"labels":[{"id":196992743,"node_id":"MDU6TGFiZWwxOTY5OTI3NDM=","url":"https://api.github.com/repos/prestodb/presto/labels/CLA%20Signed","name":"CLA Signed","color":"ededed","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2018-11-13T06:58:50Z","updated_at":"2018-12-03T07:56:17Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/prestodb/presto/pulls/11910","html_url":"https://github.com/prestodb/presto/pull/11910","diff_url":"https://github.com/prestodb/presto/pull/11910.diff","patch_url":"https://github.com/prestodb/presto/pull/11910.patch"},"body":"This commit adds instrumentation about the spilled data size in the\r\nquery summary and the web UI.\r\n\r\nThis implementation adds an operator stat that is rolled up\r\nto the query level. I followed the approach in the comment:\r\nhttps://github.com/prestodb/presto/pull/8010#issuecomment-314844081\r\n\r\nNote that I have not included the React-generated query JavaScript. I can\r\ninclude that also."},"comment":{"url":"https://api.github.com/repos/prestodb/presto/issues/comments/443619885","html_url":"https://github.com/prestodb/presto/pull/11910#issuecomment-443619885","issue_url":"https://api.github.com/repos/prestodb/presto/issues/11910","id":443619885,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0MzYxOTg4NQ==","user":{"login":"electrum","id":9230,"node_id":"MDQ6VXNlcjkyMzA=","avatar_url":"https://avatars1.githubusercontent.com/u/9230?v=4","gravatar_id":"","url":"https://api.github.com/users/electrum","html_url":"https://github.com/electrum","followers_url":"https://api.github.com/users/electrum/followers","following_url":"https://api.github.com/users/electrum/following{/other_user}","gists_url":"https://api.github.com/users/electrum/gists{/gist_id}","starred_url":"https://api.github.com/users/electrum/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/electrum/subscriptions","organizations_url":"https://api.github.com/users/electrum/orgs","repos_url":"https://api.github.com/users/electrum/repos","events_url":"https://api.github.com/users/electrum/events{/privacy}","received_events_url":"https://api.github.com/users/electrum/received_events","type":"User","site_admin":false},"created_at":"2018-12-03T07:56:17Z","updated_at":"2018-12-03T07:56:17Z","author_association":"CONTRIBUTOR","body":"I agree with @raghavsethi that it's probably cleaner to only print the value in the CLI and UI if it is greater than zero. Though for \"physical written bytes\" (and possibly others), we always show it in the UI, even when it's not relevant."}}
{"actor":{"display_login":"electrum"}}