{"action":"created","issue":{"url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/24","repository_url":"https://api.github.com/repos/Jaeger2305/werewolves-site","labels_url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/24/comments","events_url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/24/events","html_url":"https://github.com/Jaeger2305/werewolves-site/issues/24","id":145025053,"number":24,"title":"Changes to redis are not atomic","user":{"login":"Jaeger2305","id":3169945,"avatar_url":"https://avatars.githubusercontent.com/u/3169945?v=3","gravatar_id":"","url":"https://api.github.com/users/Jaeger2305","html_url":"https://github.com/Jaeger2305","followers_url":"https://api.github.com/users/Jaeger2305/followers","following_url":"https://api.github.com/users/Jaeger2305/following{/other_user}","gists_url":"https://api.github.com/users/Jaeger2305/gists{/gist_id}","starred_url":"https://api.github.com/users/Jaeger2305/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Jaeger2305/subscriptions","organizations_url":"https://api.github.com/users/Jaeger2305/orgs","repos_url":"https://api.github.com/users/Jaeger2305/repos","events_url":"https://api.github.com/users/Jaeger2305/events{/privacy}","received_events_url":"https://api.github.com/users/Jaeger2305/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Jaeger2305/werewolves-site/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/Jaeger2305/werewolves-site/labels/Low%20priority","name":"Low priority","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-03-31T21:36:06Z","updated_at":"2016-07-03T00:44:41Z","closed_at":null,"body":"This only becomes an issue with data that can be modified from multiple sources simultaneously, which should actually be pretty unlikely in a turn based game (until it's the hidden, complex bug after 6 months of coding...). Scenarios where this comes into play would currently be limited to player's joining/leaving.\r\n\r\nWorth being aware of though for future features.\r\n\r\nSee\r\nhttps://github.com/SPSCommerce/redlock-py\r\nand\r\nhttp://redis.io/topics/distlock\r\nfor guidelines.\r\n\r\nInspired by #11."},"comment":{"url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/comments/230129055","html_url":"https://github.com/Jaeger2305/werewolves-site/issues/24#issuecomment-230129055","issue_url":"https://api.github.com/repos/Jaeger2305/werewolves-site/issues/24","id":230129055,"user":{"login":"Jaeger2305","id":3169945,"avatar_url":"https://avatars.githubusercontent.com/u/3169945?v=3","gravatar_id":"","url":"https://api.github.com/users/Jaeger2305","html_url":"https://github.com/Jaeger2305","followers_url":"https://api.github.com/users/Jaeger2305/followers","following_url":"https://api.github.com/users/Jaeger2305/following{/other_user}","gists_url":"https://api.github.com/users/Jaeger2305/gists{/gist_id}","starred_url":"https://api.github.com/users/Jaeger2305/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Jaeger2305/subscriptions","organizations_url":"https://api.github.com/users/Jaeger2305/orgs","repos_url":"https://api.github.com/users/Jaeger2305/repos","events_url":"https://api.github.com/users/Jaeger2305/events{/privacy}","received_events_url":"https://api.github.com/users/Jaeger2305/received_events","type":"User","site_admin":false},"created_at":"2016-07-03T00:44:41Z","updated_at":"2016-07-03T00:44:41Z","body":"Still a potential issue despite fixing #11. One way to help reduce the chance of conflicts is to filter what is saved in Class.save() functions.\r\n\r\nAlternatively or additionally, you could flag a record (simulating a lock) and add a delay/callback if the lock is held by someone else.\r\n\r\nOn top of this, you could checkout a copy before your commit, then compare the current status to before and after your commit, and attempt to merge any conflicts (or at least error loudly)\r\n\r\nVery low priority as far as I'm concerned!"}}
{"actor":{"display_login":"Jaeger2305"}}