{"action":"created","issue":{"url":"https://api.github.com/repos/scalaz/scalaz/issues/1071","repository_url":"https://api.github.com/repos/scalaz/scalaz","labels_url":"https://api.github.com/repos/scalaz/scalaz/issues/1071/labels{/name}","comments_url":"https://api.github.com/repos/scalaz/scalaz/issues/1071/comments","events_url":"https://api.github.com/repos/scalaz/scalaz/issues/1071/events","html_url":"https://github.com/scalaz/scalaz/pull/1071","id":120130741,"number":1071,"title":"Make Applicative instance for Future sequential","user":{"login":"markus1189","id":591567,"avatar_url":"https://avatars.githubusercontent.com/u/591567?v=3","gravatar_id":"","url":"https://api.github.com/users/markus1189","html_url":"https://github.com/markus1189","followers_url":"https://api.github.com/users/markus1189/followers","following_url":"https://api.github.com/users/markus1189/following{/other_user}","gists_url":"https://api.github.com/users/markus1189/gists{/gist_id}","starred_url":"https://api.github.com/users/markus1189/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/markus1189/subscriptions","organizations_url":"https://api.github.com/users/markus1189/orgs","repos_url":"https://api.github.com/users/markus1189/repos","events_url":"https://api.github.com/users/markus1189/events{/privacy}","received_events_url":"https://api.github.com/users/markus1189/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":30,"created_at":"2015-12-03T09:51:56Z","updated_at":"2016-02-21T17:32:52Z","closed_at":"2016-02-20T18:47:16Z","pull_request":{"url":"https://api.github.com/repos/scalaz/scalaz/pulls/1071","html_url":"https://github.com/scalaz/scalaz/pull/1071","diff_url":"https://github.com/scalaz/scalaz/pull/1071.diff","patch_url":"https://github.com/scalaz/scalaz/pull/1071.patch"},"body":"DISCLAIMER: I was unsure if I should do this pr at all, because I profit from the parallel applicative instance of Future currently, but I think it should at least be mentioned ;)\r\n\r\nThe issue is that in the definition of `ap` for the standard `scala.concurrent.Future` it uses the `Future.zip` method which runs in parallel (introduced here 4fc139114f38cb408e56e9cdb00d0307e2b089d4).  This clashes with the rule that the `ap` definition should be equivalent to the definition which uses `flatMap` if there also is a `Monad` instance. \r\n\r\nMaybe you are all aware of this fact and decided that it is nevertheless a beneficial evil, so take it or leave it ;)\r\n\r\n-- 24pullrequests\r\n[![](http://24pullrequests.com/assets/logo-8a77737f86fec8def19a1c9a605c9841dbd44e59f243ed3bf64bbdf3214d6fa1.png)](http://24pullrequests.com/)"},"comment":{"url":"https://api.github.com/repos/scalaz/scalaz/issues/comments/186864718","html_url":"https://github.com/scalaz/scalaz/pull/1071#issuecomment-186864718","issue_url":"https://api.github.com/repos/scalaz/scalaz/issues/1071","id":186864718,"user":{"login":"LeifW","id":28711,"avatar_url":"https://avatars.githubusercontent.com/u/28711?v=3","gravatar_id":"","url":"https://api.github.com/users/LeifW","html_url":"https://github.com/LeifW","followers_url":"https://api.github.com/users/LeifW/followers","following_url":"https://api.github.com/users/LeifW/following{/other_user}","gists_url":"https://api.github.com/users/LeifW/gists{/gist_id}","starred_url":"https://api.github.com/users/LeifW/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LeifW/subscriptions","organizations_url":"https://api.github.com/users/LeifW/orgs","repos_url":"https://api.github.com/users/LeifW/repos","events_url":"https://api.github.com/users/LeifW/events{/privacy}","received_events_url":"https://api.github.com/users/LeifW/received_events","type":"User","site_admin":false},"created_at":"2016-02-21T17:32:52Z","updated_at":"2016-02-21T17:32:52Z","body":"Though, it seems that `// override for actual parallel execution` comment has been a lie this whole time. :("}}