{"action":"created","issue":{"url":"https://api.github.com/repos/prestodb/presto/issues/12104","repository_url":"https://api.github.com/repos/prestodb/presto","labels_url":"https://api.github.com/repos/prestodb/presto/issues/12104/labels{/name}","comments_url":"https://api.github.com/repos/prestodb/presto/issues/12104/comments","events_url":"https://api.github.com/repos/prestodb/presto/issues/12104/events","html_url":"https://github.com/prestodb/presto/pull/12104","id":392390108,"node_id":"MDExOlB1bGxSZXF1ZXN0MjM5Njc0NTI1","number":12104,"title":"Enable char predicates to be pushed down.","user":{"login":"cbanek","id":20155748,"node_id":"MDQ6VXNlcjIwMTU1NzQ4","avatar_url":"https://avatars3.githubusercontent.com/u/20155748?v=4","gravatar_id":"","url":"https://api.github.com/users/cbanek","html_url":"https://github.com/cbanek","followers_url":"https://api.github.com/users/cbanek/followers","following_url":"https://api.github.com/users/cbanek/following{/other_user}","gists_url":"https://api.github.com/users/cbanek/gists{/gist_id}","starred_url":"https://api.github.com/users/cbanek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cbanek/subscriptions","organizations_url":"https://api.github.com/users/cbanek/orgs","repos_url":"https://api.github.com/users/cbanek/repos","events_url":"https://api.github.com/users/cbanek/events{/privacy}","received_events_url":"https://api.github.com/users/cbanek/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":7,"created_at":"2018-12-19T00:18:18Z","updated_at":"2018-12-26T19:36:24Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/prestodb/presto/pulls/12104","html_url":"https://github.com/prestodb/presto/pull/12104","diff_url":"https://github.com/prestodb/presto/pull/12104.diff","patch_url":"https://github.com/prestodb/presto/pull/12104.patch"},"body":"When running queries with presto, if the underlying\r\ntable is of type varchar, the predicate gets pushed.\r\n\r\nIf it's of type char, it will NOT get pushed, which\r\ncauses presto to get the whole table, and then filter\r\nlocally, which is a big pain (and bad performance).\r\n\r\nUsing Presto PR #4842 as a guide, I've added support\r\nfor char types to also be passed down.  Unit tests\r\nalso updated for the type following the pattern.\r\nAlso integration tested against a mysql-like backend."},"comment":{"url":"https://api.github.com/repos/prestodb/presto/issues/comments/450014351","html_url":"https://github.com/prestodb/presto/pull/12104#issuecomment-450014351","issue_url":"https://api.github.com/repos/prestodb/presto/issues/12104","id":450014351,"node_id":"MDEyOklzc3VlQ29tbWVudDQ1MDAxNDM1MQ==","user":{"login":"cbanek","id":20155748,"node_id":"MDQ6VXNlcjIwMTU1NzQ4","avatar_url":"https://avatars3.githubusercontent.com/u/20155748?v=4","gravatar_id":"","url":"https://api.github.com/users/cbanek","html_url":"https://github.com/cbanek","followers_url":"https://api.github.com/users/cbanek/followers","following_url":"https://api.github.com/users/cbanek/following{/other_user}","gists_url":"https://api.github.com/users/cbanek/gists{/gist_id}","starred_url":"https://api.github.com/users/cbanek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cbanek/subscriptions","organizations_url":"https://api.github.com/users/cbanek/orgs","repos_url":"https://api.github.com/users/cbanek/repos","events_url":"https://api.github.com/users/cbanek/events{/privacy}","received_events_url":"https://api.github.com/users/cbanek/received_events","type":"User","site_admin":false},"created_at":"2018-12-26T19:36:24Z","updated_at":"2018-12-26T19:36:24Z","author_association":"NONE","body":"Hope everyone had a good Christmas!  I've made the requested changes and I think this is ready for another round by @findepi and @sopel39 "}}
{"actor":{"display_login":"cbanek"}}