)]}'
{"swift/common/db.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"a464014cda1fac556bc3c172507d40f2188d8822","unresolved":false,"context_lines":[{"line_number":876,"context_line":"                self.quarantine(\"missing row in %s_stat table\" %"},{"line_number":877,"context_line":"                                self.db_type)"},{"line_number":878,"context_line":"            md \u003d md[0]"},{"line_number":879,"context_line":"            if md:"},{"line_number":880,"context_line":"                md \u003d json.loads(md)"},{"line_number":881,"context_line":"                keys_to_delete \u003d []"},{"line_number":882,"context_line":"                for key, (value, value_timestamp) in md.items():"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa0c359_ef34d44c","line":879,"updated":"2018-02-06 20:22:50.000000000","message":"nit: with the `if not md:` above, this looks a little funny. I wonder if we should maybe rename our fetchone() result to be \u0027row\u0027 to be more clear.","commit_id":"1ea65339cb10851ba451e7918acabd6f2db3d442"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"7ad58c69621ec1f8cc38d19e8cf58950e2250826","unresolved":false,"context_lines":[{"line_number":875,"context_line":"            if not row:"},{"line_number":876,"context_line":"                self.quarantine(\"missing row in %s_stat table\" %"},{"line_number":877,"context_line":"                                self.db_type)"},{"line_number":878,"context_line":"            md \u003d row[0]"},{"line_number":879,"context_line":"            if md:"},{"line_number":880,"context_line":"                md \u003d json.loads(md)"},{"line_number":881,"context_line":"                keys_to_delete \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"3fa0c359_2c185f7f","line":878,"updated":"2018-02-08 00:26:09.000000000","message":"seeing as we have pretty much the same thing written 3 times here, surely this is a good candidate for a refactor?\n\nSomething like: http://paste.openstack.org/show/665484/","commit_id":"bfe52a2e3562243b1b488e339a965642d846a30e"}]}
