{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/orleans/issues/985","labels_url":"https://api.github.com/repos/dotnet/orleans/issues/985/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/orleans/issues/985/comments","events_url":"https://api.github.com/repos/dotnet/orleans/issues/985/events","html_url":"https://github.com/dotnet/orleans/pull/985","id":114923057,"number":985,"title":"MSSQL MBR - Updating IAmAlive shouldn't update ETag","user":{"login":"shayhatsor","id":12084632,"avatar_url":"https://avatars.githubusercontent.com/u/12084632?v=3","gravatar_id":"","url":"https://api.github.com/users/shayhatsor","html_url":"https://github.com/shayhatsor","followers_url":"https://api.github.com/users/shayhatsor/followers","following_url":"https://api.github.com/users/shayhatsor/following{/other_user}","gists_url":"https://api.github.com/users/shayhatsor/gists{/gist_id}","starred_url":"https://api.github.com/users/shayhatsor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shayhatsor/subscriptions","organizations_url":"https://api.github.com/users/shayhatsor/orgs","repos_url":"https://api.github.com/users/shayhatsor/repos","events_url":"https://api.github.com/users/shayhatsor/events{/privacy}","received_events_url":"https://api.github.com/users/shayhatsor/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dotnet/orleans/labels/cla-already-signed","name":"cla-already-signed","color":"009800"},{"url":"https://api.github.com/repos/dotnet/orleans/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":11,"created_at":"2015-11-03T22:16:35Z","updated_at":"2015-11-05T01:29:59Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dotnet/orleans/pulls/985","html_url":"https://github.com/dotnet/orleans/pull/985","diff_url":"https://github.com/dotnet/orleans/pull/985.diff","patch_url":"https://github.com/dotnet/orleans/pull/985.patch"},"body":"I'm currently completing the MySQL ADO Invariant implementation in Orleans.\r\nThis includes creating a complete MySQL setup script as available in MSSQL and adding relevant tests.\r\nWhile converting *CreateOrleansTables_SqlServer.sql* to MySQL I came across what IMHO is a bug.\r\nIn current implementation, the **UpdateIAmAlivetimeKey** script updates the **ETag** field.\r\nThe **IAmAliveTime** is a purely informative field and isn't a part of the membership protocol.\r\nThis will cause silos to unnecessarily fail periodically when trying to update the membership table.\r\nMy solution is to use the MSSQL 2000 implementation for all versions of MSSQL, with the minor change of not updating the **ETag** in **UpdateIAmAlivetimeKey**.\r\n@veikkoeeva what do you think?"},"comment":{"url":"https://api.github.com/repos/dotnet/orleans/issues/comments/153924935","html_url":"https://github.com/dotnet/orleans/pull/985#issuecomment-153924935","issue_url":"https://api.github.com/repos/dotnet/orleans/issues/985","id":153924935,"user":{"login":"sergeybykov","id":8248806,"avatar_url":"https://avatars.githubusercontent.com/u/8248806?v=3","gravatar_id":"","url":"https://api.github.com/users/sergeybykov","html_url":"https://github.com/sergeybykov","followers_url":"https://api.github.com/users/sergeybykov/followers","following_url":"https://api.github.com/users/sergeybykov/following{/other_user}","gists_url":"https://api.github.com/users/sergeybykov/gists{/gist_id}","starred_url":"https://api.github.com/users/sergeybykov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sergeybykov/subscriptions","organizations_url":"https://api.github.com/users/sergeybykov/orgs","repos_url":"https://api.github.com/users/sergeybykov/repos","events_url":"https://api.github.com/users/sergeybykov/events{/privacy}","received_events_url":"https://api.github.com/users/sergeybykov/received_events","type":"User","site_admin":false},"created_at":"2015-11-05T01:29:59Z","updated_at":"2015-11-05T01:29:59Z","body":"Is this ready to be merged?"}}