{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/13319","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/13319/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/13319/comments","events_url":"https://api.github.com/repos/apache/spark/issues/13319/events","html_url":"https://github.com/apache/spark/pull/13319","id":156914428,"number":13319,"title":"[SPARK-15552][SQL] Remove unnecessary private[sql] methods in SparkSession","user":{"login":"rxin","id":323388,"avatar_url":"https://avatars.githubusercontent.com/u/323388?v=3","gravatar_id":"","url":"https://api.github.com/users/rxin","html_url":"https://github.com/rxin","followers_url":"https://api.github.com/users/rxin/followers","following_url":"https://api.github.com/users/rxin/following{/other_user}","gists_url":"https://api.github.com/users/rxin/gists{/gist_id}","starred_url":"https://api.github.com/users/rxin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rxin/subscriptions","organizations_url":"https://api.github.com/users/rxin/orgs","repos_url":"https://api.github.com/users/rxin/repos","events_url":"https://api.github.com/users/rxin/events{/privacy}","received_events_url":"https://api.github.com/users/rxin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2016-05-26T06:37:00Z","updated_at":"2016-05-26T17:53:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/13319","html_url":"https://github.com/apache/spark/pull/13319","diff_url":"https://github.com/apache/spark/pull/13319.diff","patch_url":"https://github.com/apache/spark/pull/13319.patch"},"body":"## What changes were proposed in this pull request?\r\nSparkSession has a list of unnecessary private[sql] methods. These methods cause some trouble because private[sql] doesn't apply in Java. In the cases that they are easy to remove, we can simply remove them. This patch does that.\r\n\r\nAs part of this pull request, I also replaced a bunch of protected[sql] with private[sql], to tighten up visibility.\r\n\r\n## How was this patch tested?\r\nUpdated test cases to reflect the changes."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/221945237","html_url":"https://github.com/apache/spark/pull/13319#issuecomment-221945237","issue_url":"https://api.github.com/repos/apache/spark/issues/13319","id":221945237,"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":"2016-05-26T17:53:11Z","updated_at":"2016-05-26T17:53:11Z","body":"**[Test build #59389 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59389/consoleFull)** for PR 13319 at commit [`4d98beb`](https://github.com/apache/spark/commit/4d98beb5fd386f55663cb3ab5fa14a3adb19172d).\n * This patch **fails PySpark unit tests**.\n * This patch merges cleanly.\n * This patch adds no public classes."}}
{"actor":{"display_login":"SparkQA"}}