)]}'
{"swift/proxy/server.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"4b833b9364d316a22570a539f5a9fb07eebc5bb9","unresolved":false,"context_lines":[{"line_number":104,"context_line":"        except ValueError as err:"},{"line_number":105,"context_line":"            # make the message a little more useful"},{"line_number":106,"context_line":"            raise ValueError(\"Invalid read_affinity value: %r (%s)\" %"},{"line_number":107,"context_line":"                             (self.read_affinity, str(err)))"},{"line_number":108,"context_line":""},{"line_number":109,"context_line":"        self.write_affinity \u003d get(\u0027write_affinity\u0027, \u0027\u0027)"},{"line_number":110,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_a4df2564","line":107,"range":{"start_line":107,"start_character":50,"end_line":107,"end_character":53},"updated":"2017-08-17 08:10:58.000000000","message":"nit: The str() is not necessary because the format handles the conversion","commit_id":"2c7c81e8725203cf4264ab152498317c2f530335"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"4b833b9364d316a22570a539f5a9fb07eebc5bb9","unresolved":false,"context_lines":[{"line_number":113,"context_line":"        except ValueError as err:"},{"line_number":114,"context_line":"            # make the message a little more useful"},{"line_number":115,"context_line":"            raise ValueError(\"Invalid write_affinity value: %r (%s)\" %"},{"line_number":116,"context_line":"                             (self.write_affinity, str(err)))"},{"line_number":117,"context_line":"        self.write_affinity_node_count \u003d get("},{"line_number":118,"context_line":"            \u0027write_affinity_node_count\u0027, \u00272 * replicas\u0027).lower()"},{"line_number":119,"context_line":"        value \u003d self.write_affinity_node_count.split()"}],"source_content_type":"text/x-python","patch_set":2,"id":"9f436f4f_84d42149","line":116,"range":{"start_line":116,"start_character":51,"end_line":116,"end_character":54},"updated":"2017-08-17 08:10:58.000000000","message":"nit: ditto","commit_id":"2c7c81e8725203cf4264ab152498317c2f530335"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"92313cf1e618e6929d074ee2314c4e99a58c39e5","unresolved":false,"context_lines":[{"line_number":295,"context_line":"                              (label_for_policy, override))"},{"line_number":296,"context_line":"            return override"},{"line_number":297,"context_line":"        except ValueError as err:"},{"line_number":298,"context_line":"            raise ValueError(err + \u0027 for %s\u0027 % label_for_policy)"},{"line_number":299,"context_line":""},{"line_number":300,"context_line":"    def _load_per_policy_config(self, conf):"},{"line_number":301,"context_line":"        \"\"\""}],"source_content_type":"text/x-python","patch_set":3,"id":"9f436f4f_22156549","line":298,"range":{"start_line":298,"start_character":29,"end_line":298,"end_character":63},"updated":"2017-08-17 09:56:42.000000000","message":"I suggest\n\n  \u0027%s for %s\u0027 % (err, label_for_policy)","commit_id":"108b106e47a840b0a67c5827465f3d1320f7b159"}]}