{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15652","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15652/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15652/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15652/events","html_url":"https://github.com/apache/spark/pull/15652","id":185544172,"number":15652,"title":"[SPARK-16987] [None] Add spark-default.conf property to define https port for spark history server","user":{"login":"hayashidac","id":5276569,"avatar_url":"https://avatars.githubusercontent.com/u/5276569?v=3","gravatar_id":"","url":"https://api.github.com/users/hayashidac","html_url":"https://github.com/hayashidac","followers_url":"https://api.github.com/users/hayashidac/followers","following_url":"https://api.github.com/users/hayashidac/following{/other_user}","gists_url":"https://api.github.com/users/hayashidac/gists{/gist_id}","starred_url":"https://api.github.com/users/hayashidac/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hayashidac/subscriptions","organizations_url":"https://api.github.com/users/hayashidac/orgs","repos_url":"https://api.github.com/users/hayashidac/repos","events_url":"https://api.github.com/users/hayashidac/events{/privacy}","received_events_url":"https://api.github.com/users/hayashidac/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-10-27T00:23:10Z","updated_at":"2016-11-20T00:22:35Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15652","html_url":"https://github.com/apache/spark/pull/15652","diff_url":"https://github.com/apache/spark/pull/15652.diff","patch_url":"https://github.com/apache/spark/pull/15652.patch"},"body":"I created a pull request, but there is a problem.\nPort should be separated for each particular process such as history server and spark ui.\nBut user can configure spark.ssl.port with this code and when configure it, each process will try to create connections with same port and it doesn't work good.\nBecause of structure of existing function, it has impact to other functions to solve the problem.\nSo there are three answer about it.\n1. Use this fix code so user can configure a specific secure port, but spark.ssl.port doesn't work good.\n2. Don't change and close issue.\n3. We should be able to configure a specific secure port completely, so drastically change many functions.\nPlease comments. \n"},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/261749192","html_url":"https://github.com/apache/spark/pull/15652#issuecomment-261749192","issue_url":"https://api.github.com/repos/apache/spark/issues/15652","id":261749192,"user":{"login":"vanzin","id":1694083,"avatar_url":"https://avatars.githubusercontent.com/u/1694083?v=3","gravatar_id":"","url":"https://api.github.com/users/vanzin","html_url":"https://github.com/vanzin","followers_url":"https://api.github.com/users/vanzin/followers","following_url":"https://api.github.com/users/vanzin/following{/other_user}","gists_url":"https://api.github.com/users/vanzin/gists{/gist_id}","starred_url":"https://api.github.com/users/vanzin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vanzin/subscriptions","organizations_url":"https://api.github.com/users/vanzin/orgs","repos_url":"https://api.github.com/users/vanzin/repos","events_url":"https://api.github.com/users/vanzin/events{/privacy}","received_events_url":"https://api.github.com/users/vanzin/received_events","type":"User","site_admin":false},"created_at":"2016-11-20T00:22:35Z","updated_at":"2016-11-20T00:22:35Z","body":"Aside from me not liking the API change you're introducing, you need to add a unit test to make sure the feature is working. I suggest taking the scenario I described in a previous comment (https://github.com/apache/spark/pull/15652#discussion_r88137680) and turning it into a unit test."}}
{"actor":{"display_login":"vanzin"}}