{"action":"created","issue":{"url":"https://api.github.com/repos/cmu-db/peloton/issues/886","repository_url":"https://api.github.com/repos/cmu-db/peloton","labels_url":"https://api.github.com/repos/cmu-db/peloton/issues/886/labels{/name}","comments_url":"https://api.github.com/repos/cmu-db/peloton/issues/886/comments","events_url":"https://api.github.com/repos/cmu-db/peloton/issues/886/events","html_url":"https://github.com/cmu-db/peloton/pull/886","id":273001122,"number":886,"title":"Remove parser hack for SELECT without FROM","user":{"login":"hzxa21","id":5518566,"avatar_url":"https://avatars1.githubusercontent.com/u/5518566?v=4","gravatar_id":"","url":"https://api.github.com/users/hzxa21","html_url":"https://github.com/hzxa21","followers_url":"https://api.github.com/users/hzxa21/followers","following_url":"https://api.github.com/users/hzxa21/following{/other_user}","gists_url":"https://api.github.com/users/hzxa21/gists{/gist_id}","starred_url":"https://api.github.com/users/hzxa21/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hzxa21/subscriptions","organizations_url":"https://api.github.com/users/hzxa21/orgs","repos_url":"https://api.github.com/users/hzxa21/repos","events_url":"https://api.github.com/users/hzxa21/events{/privacy}","received_events_url":"https://api.github.com/users/hzxa21/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2017-11-10T17:13:19Z","updated_at":"2017-11-16T03:56:11Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/cmu-db/peloton/pulls/886","html_url":"https://github.com/cmu-db/peloton/pull/886","diff_url":"https://github.com/cmu-db/peloton/pull/886.diff","patch_url":"https://github.com/cmu-db/peloton/pull/886.patch"},"body":"This PR fixes SELECT without FROM by removing the hack in the parser and conducting the check in the binder. We should get this merged in before adding support for new built-in SQL functions.\r\n\r\nThis PR also enables binder to actually bind function expression. Previously the binding is done in the optimizer since we didn't have a binder then :("},"comment":{"url":"https://api.github.com/repos/cmu-db/peloton/issues/comments/344808636","html_url":"https://github.com/cmu-db/peloton/pull/886#issuecomment-344808636","issue_url":"https://api.github.com/repos/cmu-db/peloton/issues/886","id":344808636,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars1.githubusercontent.com/u/2354108?v=4","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2017-11-16T03:56:11Z","updated_at":"2017-11-16T03:56:11Z","author_association":"NONE","body":"\n[![Coverage Status](https://coveralls.io/builds/14227670/badge)](https://coveralls.io/builds/14227670)\n\nCoverage decreased (-0.1%) to 75.085% when pulling **970d3c2b3a994a9c6a0a8f50c4c5ed374a2bd582 on hzxa21:hackathon_fix** into **b4a974739f25f4bd4f1919c8c78ed779b0a07c9e on cmu-db:master**.\n"}}
{"actor":{"display_login":"coveralls"}}