{"action":"created","issue":{"url":"https://api.github.com/repos/slamdata/slamengine/issues/761","labels_url":"https://api.github.com/repos/slamdata/slamengine/issues/761/labels{/name}","comments_url":"https://api.github.com/repos/slamdata/slamengine/issues/761/comments","events_url":"https://api.github.com/repos/slamdata/slamengine/issues/761/events","html_url":"https://github.com/slamdata/slamengine/issues/761","id":84828617,"number":761,"title":"issue when searching files with _id columns","user":{"login":"garyb","id":693642,"avatar_url":"https://avatars.githubusercontent.com/u/693642?v=3","gravatar_id":"","url":"https://api.github.com/users/garyb","html_url":"https://github.com/garyb","followers_url":"https://api.github.com/users/garyb/followers","following_url":"https://api.github.com/users/garyb/following{/other_user}","gists_url":"https://api.github.com/users/garyb/gists{/gist_id}","starred_url":"https://api.github.com/users/garyb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garyb/subscriptions","organizations_url":"https://api.github.com/users/garyb/orgs","repos_url":"https://api.github.com/users/garyb/repos","events_url":"https://api.github.com/users/garyb/events{/privacy}","received_events_url":"https://api.github.com/users/garyb/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/slamdata/slamengine/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/slamdata/slamengine/labels/ready","name":"ready","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/slamdata/slamengine/milestones/8","html_url":"https://github.com/slamdata/slamengine/milestones/2.1","labels_url":"https://api.github.com/repos/slamdata/slamengine/milestones/8/labels","id":947148,"number":8,"title":"2.1","description":"","creator":{"login":"jdegoes","id":156745,"avatar_url":"https://avatars.githubusercontent.com/u/156745?v=3","gravatar_id":"","url":"https://api.github.com/users/jdegoes","html_url":"https://github.com/jdegoes","followers_url":"https://api.github.com/users/jdegoes/followers","following_url":"https://api.github.com/users/jdegoes/following{/other_user}","gists_url":"https://api.github.com/users/jdegoes/gists{/gist_id}","starred_url":"https://api.github.com/users/jdegoes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jdegoes/subscriptions","organizations_url":"https://api.github.com/users/jdegoes/orgs","repos_url":"https://api.github.com/users/jdegoes/repos","events_url":"https://api.github.com/users/jdegoes/events{/privacy}","received_events_url":"https://api.github.com/users/jdegoes/received_events","type":"User","site_admin":false},"open_issues":21,"closed_issues":4,"state":"open","created_at":"2015-01-22T17:02:56Z","updated_at":"2015-06-24T19:09:20Z","due_on":"2015-06-29T06:00:00Z","closed_at":null},"comments":4,"created_at":"2015-06-04T00:07:16Z","updated_at":"2015-06-25T19:48:41Z","closed_at":null,"body":"If I have a query like `SELECT * FROM \"/slamengine-test-01/zips\" WHERE true`, and then search the results, the engine returns an error:\r\n\r\n```\r\nAn error occurred during evaluation: com.mongodb.MongoCommandException: Command failed with error 16007: 'exception: can't convert from BSON type OID to String' on server ds045089.mongolab.com:45089. The full response is { \"errmsg\" : \"exception: can't convert from BSON type OID to String\", \"code\" : 16007, \"ok\" : 0.0 }\r\n```\r\n\r\nAvoiding selecting the `_id` field in the original query fixes this."},"comment":{"url":"https://api.github.com/repos/slamdata/slamengine/issues/comments/115375122","html_url":"https://github.com/slamdata/slamengine/issues/761#issuecomment-115375122","issue_url":"https://api.github.com/repos/slamdata/slamengine/issues/761","id":115375122,"user":{"login":"sellout","id":33031,"avatar_url":"https://avatars.githubusercontent.com/u/33031?v=3","gravatar_id":"","url":"https://api.github.com/users/sellout","html_url":"https://github.com/sellout","followers_url":"https://api.github.com/users/sellout/followers","following_url":"https://api.github.com/users/sellout/following{/other_user}","gists_url":"https://api.github.com/users/sellout/gists{/gist_id}","starred_url":"https://api.github.com/users/sellout/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sellout/subscriptions","organizations_url":"https://api.github.com/users/sellout/orgs","repos_url":"https://api.github.com/users/sellout/repos","events_url":"https://api.github.com/users/sellout/events{/privacy}","received_events_url":"https://api.github.com/users/sellout/received_events","type":"User","site_admin":false},"created_at":"2015-06-25T19:48:40Z","updated_at":"2015-06-25T19:48:40Z","body":"Right – but many operations just return null when applied to the \"wrong\" types, and we don't want to do extra work to protect those. But yeah, generalizing our ExprOp type checks beyond the current two special cases is necessary.\r\n\r\nAs far as null vs undefined, we unfortunately don't always have control over that."}}