{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/5372","labels_url":"https://api.github.com/repos/apache/spark/issues/5372/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/5372/comments","events_url":"https://api.github.com/repos/apache/spark/issues/5372/events","html_url":"https://github.com/apache/spark/pull/5372","id":66482383,"number":5372,"title":"[SPARK-6716] Change SparkContext.DRIVER_IDENTIFIER from <driver> to driver","user":{"login":"JoshRosen","id":50748,"avatar_url":"https://avatars.githubusercontent.com/u/50748?v=3","gravatar_id":"","url":"https://api.github.com/users/JoshRosen","html_url":"https://github.com/JoshRosen","followers_url":"https://api.github.com/users/JoshRosen/followers","following_url":"https://api.github.com/users/JoshRosen/following{/other_user}","gists_url":"https://api.github.com/users/JoshRosen/gists{/gist_id}","starred_url":"https://api.github.com/users/JoshRosen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JoshRosen/subscriptions","organizations_url":"https://api.github.com/users/JoshRosen/orgs","repos_url":"https://api.github.com/users/JoshRosen/repos","events_url":"https://api.github.com/users/JoshRosen/events{/privacy}","received_events_url":"https://api.github.com/users/JoshRosen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-04-05T21:07:09Z","updated_at":"2015-04-05T22:35:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/5372","html_url":"https://github.com/apache/spark/pull/5372","diff_url":"https://github.com/apache/spark/pull/5372.diff","patch_url":"https://github.com/apache/spark/pull/5372.patch"},"body":"Currently, the driver's executorId is set to `<driver>`. This choice of ID was present in older Spark versions, but it has started to cause problems now that executorIds are used in more contexts, such as Ganglia metric names or driver thread-dump links the web UI. The angle brackets must be escaped when embedding this ID in XML or as part of URLs and this has led to multiple problems:\r\n\r\n- https://issues.apache.org/jira/browse/SPARK-6484\r\n- https://issues.apache.org/jira/browse/SPARK-4313\r\n\r\nThe simplest solution seems to be to change this id to something that does not contain any special characters, such as `driver`.\r\n\r\nI'm not sure whether we can perform this change in a patch release, since this ID may be considered a stable API by metrics users, but it's probably okay to do this in a major release as long as we document it in the release notes."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/89866536","html_url":"https://github.com/apache/spark/pull/5372#issuecomment-89866536","issue_url":"https://api.github.com/repos/apache/spark/issues/5372","id":89866536,"user":{"login":"AmplabJenkins","id":4031291,"avatar_url":"https://avatars.githubusercontent.com/u/4031291?v=3","gravatar_id":"","url":"https://api.github.com/users/AmplabJenkins","html_url":"https://github.com/AmplabJenkins","followers_url":"https://api.github.com/users/AmplabJenkins/followers","following_url":"https://api.github.com/users/AmplabJenkins/following{/other_user}","gists_url":"https://api.github.com/users/AmplabJenkins/gists{/gist_id}","starred_url":"https://api.github.com/users/AmplabJenkins/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AmplabJenkins/subscriptions","organizations_url":"https://api.github.com/users/AmplabJenkins/orgs","repos_url":"https://api.github.com/users/AmplabJenkins/repos","events_url":"https://api.github.com/users/AmplabJenkins/events{/privacy}","received_events_url":"https://api.github.com/users/AmplabJenkins/received_events","type":"User","site_admin":false},"created_at":"2015-04-05T22:35:48Z","updated_at":"2015-04-05T22:35:48Z","body":"Test PASSed.\nRefer to this link for build results (access rights to CI server needed): \nhttps://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29732/\nTest PASSed."}}