{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/19981","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/19981/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/19981/comments","events_url":"https://api.github.com/repos/apache/spark/issues/19981/events","html_url":"https://github.com/apache/spark/pull/19981","id":282182448,"number":19981,"title":"[SPARK-22786][SQL] only use AppStatusPlugin in history server","user":{"login":"cloud-fan","id":3182036,"avatar_url":"https://avatars3.githubusercontent.com/u/3182036?v=4","gravatar_id":"","url":"https://api.github.com/users/cloud-fan","html_url":"https://github.com/cloud-fan","followers_url":"https://api.github.com/users/cloud-fan/followers","following_url":"https://api.github.com/users/cloud-fan/following{/other_user}","gists_url":"https://api.github.com/users/cloud-fan/gists{/gist_id}","starred_url":"https://api.github.com/users/cloud-fan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cloud-fan/subscriptions","organizations_url":"https://api.github.com/users/cloud-fan/orgs","repos_url":"https://api.github.com/users/cloud-fan/repos","events_url":"https://api.github.com/users/cloud-fan/events{/privacy}","received_events_url":"https://api.github.com/users/cloud-fan/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":29,"created_at":"2017-12-14T17:37:55Z","updated_at":"2017-12-19T04:42:47Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/19981","html_url":"https://github.com/apache/spark/pull/19981","diff_url":"https://github.com/apache/spark/pull/19981.diff","patch_url":"https://github.com/apache/spark/pull/19981.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nIn https://github.com/apache/spark/pull/19681 we introduced a new interface called `AppStatusPlugin`, to register listeners and set up the UI for both live and history UI.\r\n\r\nHowever I think it's an overkill for live UI. For example, we should not register `SQLListener` if users are not using SQL functions. Previously we register the `SQLListener` and set up SQL tab when `SparkSession` is firstly created, which indicates users are going to use SQL functions. But in #19681 , we register the SQL functions during `SparkContext` creation. The same thing should apply to streaming too.\r\n\r\nI think we should keep the previous behavior, and only use this new interface for history server.\r\n\r\nTo reflect this change, I also rename the new interface to `SparkHistoryUIPlugin`\r\n\r\nThis PR also refines the tests for sql listener.\r\n\r\n## How was this patch tested?\r\n\r\nexisting tests"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/352638238","html_url":"https://github.com/apache/spark/pull/19981#issuecomment-352638238","issue_url":"https://api.github.com/repos/apache/spark/issues/19981","id":352638238,"user":{"login":"SparkQA","id":8108735,"avatar_url":"https://avatars1.githubusercontent.com/u/8108735?v=4","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":"2017-12-19T04:42:47Z","updated_at":"2017-12-19T04:42:47Z","author_association":"NONE","body":"**[Test build #85091 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85091/testReport)** for PR 19981 at commit [`60421ac`](https://github.com/apache/spark/commit/60421acf5c652731d64ffad013bd4c179d6401b5)."}}
{"actor":{"display_login":"SparkQA"}}