{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/6831","labels_url":"https://api.github.com/repos/apache/spark/issues/6831/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/6831/comments","events_url":"https://api.github.com/repos/apache/spark/issues/6831/events","html_url":"https://github.com/apache/spark/pull/6831","id":88470014,"number":6831,"title":"[SPARK][SQL][Minor]reuse-typeConvert when convert Seq[Row] to CatalystType","user":{"login":"lianhuiwang","id":545478,"avatar_url":"https://avatars.githubusercontent.com/u/545478?v=3","gravatar_id":"","url":"https://api.github.com/users/lianhuiwang","html_url":"https://github.com/lianhuiwang","followers_url":"https://api.github.com/users/lianhuiwang/followers","following_url":"https://api.github.com/users/lianhuiwang/following{/other_user}","gists_url":"https://api.github.com/users/lianhuiwang/gists{/gist_id}","starred_url":"https://api.github.com/users/lianhuiwang/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lianhuiwang/subscriptions","organizations_url":"https://api.github.com/users/lianhuiwang/orgs","repos_url":"https://api.github.com/users/lianhuiwang/repos","events_url":"https://api.github.com/users/lianhuiwang/events{/privacy}","received_events_url":"https://api.github.com/users/lianhuiwang/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":10,"created_at":"2015-06-15T16:07:01Z","updated_at":"2015-06-15T16:24:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/6831","html_url":"https://github.com/apache/spark/pull/6831","diff_url":"https://github.com/apache/spark/pull/6831.diff","patch_url":"https://github.com/apache/spark/pull/6831.patch"},"body":"reuse-typeConvert when convert Seq[Row] to CatalystType"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/112126646","html_url":"https://github.com/apache/spark/pull/6831#issuecomment-112126646","issue_url":"https://api.github.com/repos/apache/spark/issues/6831","id":112126646,"user":{"login":"JoshRosen","id":50748,"avatar_url":"https://avatars.githubusercontent.com/u/50748?v=3","gravatar_id":"","url":"https://api.github.com/users/JoshRosen","html_url":"https://github.com/JoshRosen","followers_url":"https://api.github.com/users/JoshRosen/followers","following_url":"https://api.github.com/users/JoshRosen/following{/other_user}","gists_url":"https://api.github.com/users/JoshRosen/gists{/gist_id}","starred_url":"https://api.github.com/users/JoshRosen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JoshRosen/subscriptions","organizations_url":"https://api.github.com/users/JoshRosen/orgs","repos_url":"https://api.github.com/users/JoshRosen/repos","events_url":"https://api.github.com/users/JoshRosen/events{/privacy}","received_events_url":"https://api.github.com/users/JoshRosen/received_events","type":"User","site_admin":false},"created_at":"2015-06-15T16:24:44Z","updated_at":"2015-06-15T16:24:44Z","body":"The high-level motivation for this change looks good to me.  One thing that you might look into is whether this patch has removed the last uses of `CatalystTypeConverters.convertToCatalyst`; if so, I think we should go a little further and just remove that method from `CatalystTypeConverters` so that people don't attempt to use that slower path in new code.  If it's still used in places that are hard to change, though, then it's fine to leave it."}}