)]}'
{"nova/db/sqlalchemy/migrate_repo/versions/147_no_service_zones.py":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d15067e9cbfcbf94572be4bcc44be1b3f7eb0429","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        result \u003d list(query.execute())"},{"line_number":100,"context_line":"        # Check that there used to be an availability_zone. It is possible to"},{"line_number":101,"context_line":"        # have no aggregates."},{"line_number":102,"context_line":"        if len(result) \u003e 0:"},{"line_number":103,"context_line":"            services.update().values("},{"line_number":104,"context_line":"                availability_zone\u003dresult[0][0]"},{"line_number":105,"context_line":"            ).where("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_ae6e412a","line":102,"updated":"2014-08-01 09:23:02.000000000","message":"nit: pythonic way of doing it is just \u0027if result:\u0027","commit_id":"6279478fbeea4a45dc9611a861fe68af910dd640"},{"author":{"_account_id":7069,"name":"Joshua Hesketh","email":"josh@nitrotech.org","username":"jhesketh"},"change_message_id":"28c37e5a5d93a6cd29155a5c977b44ce5e6e0522","unresolved":false,"context_lines":[{"line_number":99,"context_line":"        result \u003d list(query.execute())"},{"line_number":100,"context_line":"        # Check that there used to be an availability_zone. It is possible to"},{"line_number":101,"context_line":"        # have no aggregates."},{"line_number":102,"context_line":"        if len(result) \u003e 0:"},{"line_number":103,"context_line":"            services.update().values("},{"line_number":104,"context_line":"                availability_zone\u003dresult[0][0]"},{"line_number":105,"context_line":"            ).where("}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_5e9689f1","line":102,"in_reply_to":"baada198_ae6e412a","updated":"2014-08-19 07:31:00.000000000","message":"Ihar: This is more explicit about what it is checking (ie that we actually got /rows/). You\u0027re nit-picking on a patch that\u0027s been open for 8+ months with half a dozen +1\u0027s? Lets just work on being practical and getting this merged.","commit_id":"6279478fbeea4a45dc9611a861fe68af910dd640"}]}