{"action":"created","issue":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/4149","labels_url":"https://api.github.com/repos/influxdb/influxdb/issues/4149/labels{/name}","comments_url":"https://api.github.com/repos/influxdb/influxdb/issues/4149/comments","events_url":"https://api.github.com/repos/influxdb/influxdb/issues/4149/events","html_url":"https://github.com/influxdb/influxdb/pull/4149","id":107037737,"number":4149,"title":"influxql: when using derivative, check 'group by time' instead of 'where time ...'","user":{"login":"peekeri","id":1290621,"avatar_url":"https://avatars.githubusercontent.com/u/1290621?v=3","gravatar_id":"","url":"https://api.github.com/users/peekeri","html_url":"https://github.com/peekeri","followers_url":"https://api.github.com/users/peekeri/followers","following_url":"https://api.github.com/users/peekeri/following{/other_user}","gists_url":"https://api.github.com/users/peekeri/gists{/gist_id}","starred_url":"https://api.github.com/users/peekeri/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/peekeri/subscriptions","organizations_url":"https://api.github.com/users/peekeri/orgs","repos_url":"https://api.github.com/users/peekeri/repos","events_url":"https://api.github.com/users/peekeri/events{/privacy}","received_events_url":"https://api.github.com/users/peekeri/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-09-17T17:53:28Z","updated_at":"2015-09-17T19:08:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdb/influxdb/pulls/4149","html_url":"https://github.com/influxdb/influxdb/pull/4149","diff_url":"https://github.com/influxdb/influxdb/pull/4149.diff","patch_url":"https://github.com/influxdb/influxdb/pull/4149.patch"},"body":"Proposed fix for #4148.\r\n\r\nWhen using derivative, it is required that aggregate function is used as\r\nsub-call when grouping by time is used. However, AST parsing used to check\r\nif WHERE-clause contained condition with 'time'.\r\n\r\nFix this by changing check to see if groupByInterval is present."},"comment":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/comments/141191711","html_url":"https://github.com/influxdb/influxdb/pull/4149#issuecomment-141191711","issue_url":"https://api.github.com/repos/influxdb/influxdb/issues/4149","id":141191711,"user":{"login":"otoolep","id":536312,"avatar_url":"https://avatars.githubusercontent.com/u/536312?v=3","gravatar_id":"","url":"https://api.github.com/users/otoolep","html_url":"https://github.com/otoolep","followers_url":"https://api.github.com/users/otoolep/followers","following_url":"https://api.github.com/users/otoolep/following{/other_user}","gists_url":"https://api.github.com/users/otoolep/gists{/gist_id}","starred_url":"https://api.github.com/users/otoolep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/otoolep/subscriptions","organizations_url":"https://api.github.com/users/otoolep/orgs","repos_url":"https://api.github.com/users/otoolep/repos","events_url":"https://api.github.com/users/otoolep/events{/privacy}","received_events_url":"https://api.github.com/users/otoolep/received_events","type":"User","site_admin":false},"created_at":"2015-09-17T19:08:23Z","updated_at":"2015-09-17T19:08:23Z","body":"Thanks for the PR @peekeri -- you can see the test failures here:\r\n\r\nhttps://circle-artifacts.com/gh/influxdb/influxdb/5960/artifacts/0/tmp/circle-artifacts.tXeSyhP/test_logs.txt\r\n\r\nPlease be sure to run 'go test ./...' after you've made your changes, so you can see which tests need updating."}}