{"action":"created","issue":{"url":"https://api.github.com/repos/pingcap/tidb/issues/1822","repository_url":"https://api.github.com/repos/pingcap/tidb","labels_url":"https://api.github.com/repos/pingcap/tidb/issues/1822/labels{/name}","comments_url":"https://api.github.com/repos/pingcap/tidb/issues/1822/comments","events_url":"https://api.github.com/repos/pingcap/tidb/issues/1822/events","html_url":"https://github.com/pingcap/tidb/pull/1822","id":182733258,"number":1822,"title":"plan: fix a bug about doing aggregation plan.","user":{"login":"hanfei1991","id":20884546,"avatar_url":"https://avatars.githubusercontent.com/u/20884546?v=3","gravatar_id":"","url":"https://api.github.com/users/hanfei1991","html_url":"https://github.com/hanfei1991","followers_url":"https://api.github.com/users/hanfei1991/followers","following_url":"https://api.github.com/users/hanfei1991/following{/other_user}","gists_url":"https://api.github.com/users/hanfei1991/gists{/gist_id}","starred_url":"https://api.github.com/users/hanfei1991/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hanfei1991/subscriptions","organizations_url":"https://api.github.com/users/hanfei1991/orgs","repos_url":"https://api.github.com/users/hanfei1991/repos","events_url":"https://api.github.com/users/hanfei1991/events{/privacy}","received_events_url":"https://api.github.com/users/hanfei1991/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-13T09:02:53Z","updated_at":"2016-10-13T09:12:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/pingcap/tidb/pulls/1822","html_url":"https://github.com/pingcap/tidb/pull/1822","diff_url":"https://github.com/pingcap/tidb/pull/1822.diff","patch_url":"https://github.com/pingcap/tidb/pull/1822.patch"},"body":"When we do stream agg plan, the child of it may be a nil plan with max cost. At that time, enforceProperty function will raise a panic. Instead, we should handle this case in enforceProperty to make it more sound.\r\n@shenli @coocood @zimulala @XuHuaiyu @tiancaiamao PTAL"},"comment":{"url":"https://api.github.com/repos/pingcap/tidb/issues/comments/253458872","html_url":"https://github.com/pingcap/tidb/pull/1822#issuecomment-253458872","issue_url":"https://api.github.com/repos/pingcap/tidb/issues/1822","id":253458872,"user":{"login":"coocood","id":891222,"avatar_url":"https://avatars.githubusercontent.com/u/891222?v=3","gravatar_id":"","url":"https://api.github.com/users/coocood","html_url":"https://github.com/coocood","followers_url":"https://api.github.com/users/coocood/followers","following_url":"https://api.github.com/users/coocood/following{/other_user}","gists_url":"https://api.github.com/users/coocood/gists{/gist_id}","starred_url":"https://api.github.com/users/coocood/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coocood/subscriptions","organizations_url":"https://api.github.com/users/coocood/orgs","repos_url":"https://api.github.com/users/coocood/repos","events_url":"https://api.github.com/users/coocood/events{/privacy}","received_events_url":"https://api.github.com/users/coocood/received_events","type":"User","site_admin":false},"created_at":"2016-10-13T09:12:11Z","updated_at":"2016-10-13T09:12:11Z","body":"LGTM"}}
{"actor":{"display_login":"coocood"}}