{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/5477","labels_url":"https://api.github.com/repos/apache/spark/issues/5477/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/5477/comments","events_url":"https://api.github.com/repos/apache/spark/issues/5477/events","html_url":"https://github.com/apache/spark/pull/5477","id":67881468,"number":5477,"title":"[SPARK-6868][YARN] Fix broken container log link on executor page when HTTPS_ONLY.","user":{"login":"deanchen","id":350938,"avatar_url":"https://avatars.githubusercontent.com/u/350938?v=3","gravatar_id":"","url":"https://api.github.com/users/deanchen","html_url":"https://github.com/deanchen","followers_url":"https://api.github.com/users/deanchen/followers","following_url":"https://api.github.com/users/deanchen/following{/other_user}","gists_url":"https://api.github.com/users/deanchen/gists{/gist_id}","starred_url":"https://api.github.com/users/deanchen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deanchen/subscriptions","organizations_url":"https://api.github.com/users/deanchen/orgs","repos_url":"https://api.github.com/users/deanchen/repos","events_url":"https://api.github.com/users/deanchen/events{/privacy}","received_events_url":"https://api.github.com/users/deanchen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-04-12T07:13:28Z","updated_at":"2015-04-12T12:58:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/5477","html_url":"https://github.com/apache/spark/pull/5477","diff_url":"https://github.com/apache/spark/pull/5477.diff","patch_url":"https://github.com/apache/spark/pull/5477.patch"},"body":"Correct http schema in YARN container log link in Spark UI when container logs when YARN is configured to be HTTPS_ONLY.\r\n\r\nUses the same logic as the YARN jobtracker webapp. Entry point is [JobBlock](https://github.com/apache/hadoop/blob/e1109fb65608a668cd53dc324dadc6f63a74eeb9/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/v2/app/webapp/JobBlock.java#L108) and logic is in [MRWebAppUtil](https://github.com/apache/hadoop/blob/e1109fb65608a668cd53dc324dadc6f63a74eeb9/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/util/MRWebAppUtil.java#L75).\r\n\r\nI chose to migrate the logic over instead of importing MRWebAppUtil(but can update the PR to do so) since the class is designated as @private and the logic was straightforward."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/92056486","html_url":"https://github.com/apache/spark/pull/5477#issuecomment-92056486","issue_url":"https://api.github.com/repos/apache/spark/issues/5477","id":92056486,"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-04-12T12:58:11Z","updated_at":"2015-04-12T12:58:11Z","body":"  [Test build #30117 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/30117/consoleFull) for   PR 5477 at commit [`91d3090`](https://github.com/apache/spark/commit/91d30901e833de808ba05e7dfa775aa0380aad5f).\n * This patch **passes all tests**.\n * This patch merges cleanly.\n * This patch adds no public classes.\n * This patch does not change any dependencies."}}