{"action":"created","issue":{"url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/6","repository_url":"https://api.github.com/repos/Financial-Times/notifications-push","labels_url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/6/comments","events_url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/6/events","html_url":"https://github.com/Financial-Times/notifications-push/issues/6","id":142354243,"number":6,"title":"potential issue in sleeping","user":{"login":"martingartonft","id":5197575,"avatar_url":"https://avatars.githubusercontent.com/u/5197575?v=3","gravatar_id":"","url":"https://api.github.com/users/martingartonft","html_url":"https://github.com/martingartonft","followers_url":"https://api.github.com/users/martingartonft/followers","following_url":"https://api.github.com/users/martingartonft/following{/other_user}","gists_url":"https://api.github.com/users/martingartonft/gists{/gist_id}","starred_url":"https://api.github.com/users/martingartonft/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/martingartonft/subscriptions","organizations_url":"https://api.github.com/users/martingartonft/orgs","repos_url":"https://api.github.com/users/martingartonft/repos","events_url":"https://api.github.com/users/martingartonft/events{/privacy}","received_events_url":"https://api.github.com/users/martingartonft/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-21T13:53:29Z","updated_at":"2016-03-21T13:59:09Z","closed_at":null,"body":"eventDispatcher.receiveEvents has a sleep at the end.\r\n\r\nI believe this means there will always be a 30s delay between publishes, regardless of when they happened.\r\n\r\nIn other words, 10 quick publishes will not result in a 30s delay followed by 10 quick events, it will result in 30s, 1 publish, 30s, 1 publish etc.\r\n\r\nDisclaimer: I have not reproduced this behaviour - This is from code inspection alone.\r\n"},"comment":{"url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/comments/199296016","html_url":"https://github.com/Financial-Times/notifications-push/issues/6#issuecomment-199296016","issue_url":"https://api.github.com/repos/Financial-Times/notifications-push/issues/6","id":199296016,"user":{"login":"Nesze","id":3509242,"avatar_url":"https://avatars.githubusercontent.com/u/3509242?v=3","gravatar_id":"","url":"https://api.github.com/users/Nesze","html_url":"https://github.com/Nesze","followers_url":"https://api.github.com/users/Nesze/followers","following_url":"https://api.github.com/users/Nesze/following{/other_user}","gists_url":"https://api.github.com/users/Nesze/gists{/gist_id}","starred_url":"https://api.github.com/users/Nesze/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Nesze/subscriptions","organizations_url":"https://api.github.com/users/Nesze/orgs","repos_url":"https://api.github.com/users/Nesze/repos","events_url":"https://api.github.com/users/Nesze/events{/privacy}","received_events_url":"https://api.github.com/users/Nesze/received_events","type":"User","site_admin":false},"created_at":"2016-03-21T13:59:09Z","updated_at":"2016-03-21T13:59:09Z","body":"Messages are processed concurrently, so for every new message the receiveEvents functions is spawned as a separate go routine. \r\nHowever, this can result in out-of-order messages, so we probably have to fix this."}}