{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/7462","labels_url":"https://api.github.com/repos/apache/spark/issues/7462/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/7462/comments","events_url":"https://api.github.com/repos/apache/spark/issues/7462/events","html_url":"https://github.com/apache/spark/pull/7462","id":95606658,"number":7462,"title":"[SPARK-8230][SQL] Add array/map size method","user":{"login":"EntilZha","id":1382460,"avatar_url":"https://avatars.githubusercontent.com/u/1382460?v=3","gravatar_id":"","url":"https://api.github.com/users/EntilZha","html_url":"https://github.com/EntilZha","followers_url":"https://api.github.com/users/EntilZha/followers","following_url":"https://api.github.com/users/EntilZha/following{/other_user}","gists_url":"https://api.github.com/users/EntilZha/gists{/gist_id}","starred_url":"https://api.github.com/users/EntilZha/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/EntilZha/subscriptions","organizations_url":"https://api.github.com/users/EntilZha/orgs","repos_url":"https://api.github.com/users/EntilZha/repos","events_url":"https://api.github.com/users/EntilZha/events{/privacy}","received_events_url":"https://api.github.com/users/EntilZha/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-07-17T07:45:43Z","updated_at":"2015-07-20T04:51:13Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/7462","html_url":"https://github.com/apache/spark/pull/7462","diff_url":"https://github.com/apache/spark/pull/7462.diff","patch_url":"https://github.com/apache/spark/pull/7462.patch"},"body":"Pull Request for: https://issues.apache.org/jira/browse/SPARK-8230\r\n\r\nPrimary issue resolved is to implement array/map size for Spark SQL. Code is ready for review by a committer. Chen Hao is on the JIRA ticket, but I don't know his username on github, @rxin is also on JIRA ticket.\r\n\r\nThings to review:\r\n1. Where to put added functions namespace wise, they seem to be part of a few operations on collections which includes `sort_array` and `array_contains`. Hence the name given `collectionOperations.scala` and `_collection_functions` in python.\r\n2. In Python code, should it be in a `1.5.0` function array or in a collections array?\r\n3. Are there any missing methods on the `Size` case class? Looks like many of these functions have generated Java code, is that also needed in this case?\r\n4. Something else?"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/122761679","html_url":"https://github.com/apache/spark/pull/7462#issuecomment-122761679","issue_url":"https://api.github.com/repos/apache/spark/issues/7462","id":122761679,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars.githubusercontent.com/u/8108735?v=3","gravatar_id":"","url":"https://api.github.com/users/SparkQA","html_url":"https://github.com/SparkQA","followers_url":"https://api.github.com/users/SparkQA/followers","following_url":"https://api.github.com/users/SparkQA/following{/other_user}","gists_url":"https://api.github.com/users/SparkQA/gists{/gist_id}","starred_url":"https://api.github.com/users/SparkQA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SparkQA/subscriptions","organizations_url":"https://api.github.com/users/SparkQA/orgs","repos_url":"https://api.github.com/users/SparkQA/repos","events_url":"https://api.github.com/users/SparkQA/events{/privacy}","received_events_url":"https://api.github.com/users/SparkQA/received_events","type":"User","site_admin":false},"created_at":"2015-07-20T04:51:13Z","updated_at":"2015-07-20T04:51:13Z","body":"  [Test build #37799 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37799/console) for   PR 7462 at commit [`f8a34d7`](https://github.com/apache/spark/commit/f8a34d7067a5eafd553af6c6fa48e010d66d01fe).\n * This patch **fails Scala style tests**.\n * This patch **does not merge cleanly**.\n * This patch adds the following public classes _(experimental)_:\n  * `case class Size(child: Expression) extends UnaryExpression with ExpectsInputTypes `\n"}}