{"action":"created","issue":{"url":"https://api.github.com/repos/apache/drill/issues/1138","repository_url":"https://api.github.com/repos/apache/drill","labels_url":"https://api.github.com/repos/apache/drill/issues/1138/labels{/name}","comments_url":"https://api.github.com/repos/apache/drill/issues/1138/comments","events_url":"https://api.github.com/repos/apache/drill/issues/1138/events","html_url":"https://github.com/apache/drill/pull/1138","id":300975913,"number":1138,"title":"DRILL-4120: Allow implicit columns for Avro storage format","user":{"login":"vvysotskyi","id":20928429,"avatar_url":"https://avatars3.githubusercontent.com/u/20928429?v=4","gravatar_id":"","url":"https://api.github.com/users/vvysotskyi","html_url":"https://github.com/vvysotskyi","followers_url":"https://api.github.com/users/vvysotskyi/followers","following_url":"https://api.github.com/users/vvysotskyi/following{/other_user}","gists_url":"https://api.github.com/users/vvysotskyi/gists{/gist_id}","starred_url":"https://api.github.com/users/vvysotskyi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vvysotskyi/subscriptions","organizations_url":"https://api.github.com/users/vvysotskyi/orgs","repos_url":"https://api.github.com/users/vvysotskyi/repos","events_url":"https://api.github.com/users/vvysotskyi/events{/privacy}","received_events_url":"https://api.github.com/users/vvysotskyi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-02-28T10:45:55Z","updated_at":"2018-02-28T17:16:39Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/drill/pulls/1138","html_url":"https://github.com/apache/drill/pull/1138","diff_url":"https://github.com/apache/drill/pull/1138.diff","patch_url":"https://github.com/apache/drill/pull/1138.patch"},"body":"Existing implementation of `AvroDrillTabl` does not allow dynamic columns discovering. `AvroDrillTable.getRowType()` method returns `RelDataTypeImlp` instance with the list of all table columns. It forces validator to check columns from select list in `RowType` list. It makes impossible to use implicit columns.\r\n\r\nThis fix replaces the usage of `AvroDrillTable` by `DynamicDrillTable` for Avro format and also allows usage of non-existent columns in Avro tables to be consistent with other storage formats."},"comment":{"url":"https://api.github.com/repos/apache/drill/issues/comments/369312578","html_url":"https://github.com/apache/drill/pull/1138#issuecomment-369312578","issue_url":"https://api.github.com/repos/apache/drill/issues/1138","id":369312578,"user":{"login":"paul-rogers","id":3248881,"avatar_url":"https://avatars0.githubusercontent.com/u/3248881?v=4","gravatar_id":"","url":"https://api.github.com/users/paul-rogers","html_url":"https://github.com/paul-rogers","followers_url":"https://api.github.com/users/paul-rogers/followers","following_url":"https://api.github.com/users/paul-rogers/following{/other_user}","gists_url":"https://api.github.com/users/paul-rogers/gists{/gist_id}","starred_url":"https://api.github.com/users/paul-rogers/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/paul-rogers/subscriptions","organizations_url":"https://api.github.com/users/paul-rogers/orgs","repos_url":"https://api.github.com/users/paul-rogers/repos","events_url":"https://api.github.com/users/paul-rogers/events{/privacy}","received_events_url":"https://api.github.com/users/paul-rogers/received_events","type":"User","site_admin":false},"created_at":"2018-02-28T17:16:39Z","updated_at":"2018-02-28T17:16:39Z","author_association":"CONTRIBUTOR","body":"General comment: if we could move to the new scan framework; it handles implicit columns for all file-based readers. It also handles projection, missing columns, etc..."}}
{"actor":{"display_login":"paul-rogers"}}