{"action":"created","issue":{"url":"https://api.github.com/repos/apache/arrow/issues/261","repository_url":"https://api.github.com/repos/apache/arrow","labels_url":"https://api.github.com/repos/apache/arrow/issues/261/labels{/name}","comments_url":"https://api.github.com/repos/apache/arrow/issues/261/comments","events_url":"https://api.github.com/repos/apache/arrow/issues/261/events","html_url":"https://github.com/apache/arrow/pull/261","id":198065322,"number":261,"title":"Arrow-386: [Java] Respect case of struct / map field names","user":{"login":"alphalfalfa","id":1827193,"avatar_url":"https://avatars.githubusercontent.com/u/1827193?v=3","gravatar_id":"","url":"https://api.github.com/users/alphalfalfa","html_url":"https://github.com/alphalfalfa","followers_url":"https://api.github.com/users/alphalfalfa/followers","following_url":"https://api.github.com/users/alphalfalfa/following{/other_user}","gists_url":"https://api.github.com/users/alphalfalfa/gists{/gist_id}","starred_url":"https://api.github.com/users/alphalfalfa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alphalfalfa/subscriptions","organizations_url":"https://api.github.com/users/alphalfalfa/orgs","repos_url":"https://api.github.com/users/alphalfalfa/repos","events_url":"https://api.github.com/users/alphalfalfa/events{/privacy}","received_events_url":"https://api.github.com/users/alphalfalfa/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-29T20:39:03Z","updated_at":"2016-12-30T18:49:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/arrow/pulls/261","html_url":"https://github.com/apache/arrow/pull/261","diff_url":"https://github.com/apache/arrow/pull/261.diff","patch_url":"https://github.com/apache/arrow/pull/261.patch"},"body":"Changes include:\r\n- Remove all toLowerCase() calls on field names in MapWriters.java template file, so that the writers can respect case of the field names.\r\n- Use lower-case keys for internalMap in UnionVector instead of camel-case (e.g. bigInt -> bigint). p.s. I don't know what is the original purpose of using camel case here. It did not conflict because all field names are converted to lower cases in the past.\r\n- Add a simple test case of MapWriter with mixed-case field names."},"comment":{"url":"https://api.github.com/repos/apache/arrow/issues/comments/269807655","html_url":"https://github.com/apache/arrow/pull/261#issuecomment-269807655","issue_url":"https://api.github.com/repos/apache/arrow/issues/261","id":269807655,"user":{"login":"wesm","id":329591,"avatar_url":"https://avatars.githubusercontent.com/u/329591?v=3","gravatar_id":"","url":"https://api.github.com/users/wesm","html_url":"https://github.com/wesm","followers_url":"https://api.github.com/users/wesm/followers","following_url":"https://api.github.com/users/wesm/following{/other_user}","gists_url":"https://api.github.com/users/wesm/gists{/gist_id}","starred_url":"https://api.github.com/users/wesm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wesm/subscriptions","organizations_url":"https://api.github.com/users/wesm/orgs","repos_url":"https://api.github.com/users/wesm/repos","events_url":"https://api.github.com/users/wesm/events{/privacy}","received_events_url":"https://api.github.com/users/wesm/received_events","type":"User","site_admin":false},"created_at":"2016-12-30T18:49:13Z","updated_at":"2016-12-30T18:49:13Z","body":"I expect this will also show up if added to the integration tests. @julienledem @jacques-n I'm not sure the timeline for using the Arrow jars in Drill, but I presume case-sensitivity is something you'd be able to handle without too much trouble on the application side? "}}
{"actor":{"display_login":"wesm"}}