{"action":"closed","issue":{"url":"https://api.github.com/repos/confluentinc/schema-registry/issues/138","repository_url":"https://api.github.com/repos/confluentinc/schema-registry","labels_url":"https://api.github.com/repos/confluentinc/schema-registry/issues/138/labels{/name}","comments_url":"https://api.github.com/repos/confluentinc/schema-registry/issues/138/comments","events_url":"https://api.github.com/repos/confluentinc/schema-registry/issues/138/events","html_url":"https://github.com/confluentinc/schema-registry/issues/138","id":58381168,"number":138,"title":"KafkaStoreReaderThread doWork leftover TODOs","user":{"login":"granders","id":1295932,"avatar_url":"https://avatars.githubusercontent.com/u/1295932?v=3","gravatar_id":"","url":"https://api.github.com/users/granders","html_url":"https://github.com/granders","followers_url":"https://api.github.com/users/granders/followers","following_url":"https://api.github.com/users/granders/following{/other_user}","gists_url":"https://api.github.com/users/granders/gists{/gist_id}","starred_url":"https://api.github.com/users/granders/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/granders/subscriptions","organizations_url":"https://api.github.com/users/granders/orgs","repos_url":"https://api.github.com/users/granders/repos","events_url":"https://api.github.com/users/granders/events{/privacy}","received_events_url":"https://api.github.com/users/granders/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2015-02-20T17:06:27Z","updated_at":"2016-11-09T06:01:41Z","closed_at":"2016-11-09T06:01:41Z","body":"This issue replaces a couple TODOs in KafkaStoreReaderThread\n\nin doWork(), if deserialization of messageBytes fails, should we\n            \"// TODO: fail just this operation or all subsequent operations?\"\n\nin doWork(), if the application of message to the localstore fails, \n            /**\n             \\* TODO: maybe retry for a configurable amount before logging a failure?\n             \\* TODO: maybe fail all subsequent operations of the store if retries fail\n             \\* Only 2 operations make sense if this happens -\n             \\* 1. Restart the store hoping that it works subsequently\n             \\* 2. Look into the issue manually\n             */\n"}}
{"actor":{"display_login":"ewencp"}}