{"action":"created","issue":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/2838","labels_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2838/labels{/name}","comments_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2838/comments","events_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2838/events","html_url":"https://github.com/chapel-lang/chapel/pull/2838","id":113691203,"number":2838,"title":"Minor cleanup of BlockArr follower","user":{"login":"benharsh","id":1883119,"avatar_url":"https://avatars.githubusercontent.com/u/1883119?v=3","gravatar_id":"","url":"https://api.github.com/users/benharsh","html_url":"https://github.com/benharsh","followers_url":"https://api.github.com/users/benharsh/followers","following_url":"https://api.github.com/users/benharsh/following{/other_user}","gists_url":"https://api.github.com/users/benharsh/gists{/gist_id}","starred_url":"https://api.github.com/users/benharsh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benharsh/subscriptions","organizations_url":"https://api.github.com/users/benharsh/orgs","repos_url":"https://api.github.com/users/benharsh/repos","events_url":"https://api.github.com/users/benharsh/events{/privacy}","received_events_url":"https://api.github.com/users/benharsh/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-27T21:16:58Z","updated_at":"2015-10-27T23:02:11Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chapel-lang/chapel/pulls/2838","html_url":"https://github.com/chapel-lang/chapel/pull/2838","diff_url":"https://github.com/chapel-lang/chapel/pull/2838.diff","patch_url":"https://github.com/chapel-lang/chapel/pull/2838.patch"},"body":"The non-fast branch of BlockArr's follower used a small helper function to optimize for the case that the current locale owned the index. dsiAccess already provides this behavior. In the worst case, we would perform the locality check twice before handling the general case.\r\n\r\nBy simply calling dsiAccess, we can eliminate some complex and redundant code."},"comment":{"url":"https://api.github.com/repos/chapel-lang/chapel/issues/comments/151669924","html_url":"https://github.com/chapel-lang/chapel/pull/2838#issuecomment-151669924","issue_url":"https://api.github.com/repos/chapel-lang/chapel/issues/2838","id":151669924,"user":{"login":"benharsh","id":1883119,"avatar_url":"https://avatars.githubusercontent.com/u/1883119?v=3","gravatar_id":"","url":"https://api.github.com/users/benharsh","html_url":"https://github.com/benharsh","followers_url":"https://api.github.com/users/benharsh/followers","following_url":"https://api.github.com/users/benharsh/following{/other_user}","gists_url":"https://api.github.com/users/benharsh/gists{/gist_id}","starred_url":"https://api.github.com/users/benharsh/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benharsh/subscriptions","organizations_url":"https://api.github.com/users/benharsh/orgs","repos_url":"https://api.github.com/users/benharsh/repos","events_url":"https://api.github.com/users/benharsh/events{/privacy}","received_events_url":"https://api.github.com/users/benharsh/received_events","type":"User","site_admin":false},"created_at":"2015-10-27T23:02:11Z","updated_at":"2015-10-27T23:02:11Z","body":"I confirmed that global stream still performs well. Thanks!"}}