)]}'
{"zun/db/sqlalchemy/api.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"dbbd03a908405222c085cb6ec08cf970912117e3","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            return"},{"line_number":146,"context_line":"        lowername \u003d name.lower()"},{"line_number":147,"context_line":"        base_query \u003d model_query(models.Container).\\"},{"line_number":148,"context_line":"            filter(func.lower(models.Container.name) \u003d\u003d lowername)"},{"line_number":149,"context_line":"        if CONF.compute.unique_container_name_scope \u003d\u003d \u0027project\u0027:"},{"line_number":150,"context_line":"            container_with_same_name \u003d base_query.\\"},{"line_number":151,"context_line":"                filter_by(project_id\u003dcontext.project_id).\\"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_7aabbed6","line":148,"updated":"2016-11-08 20:08:41.000000000","message":"This might be my personal taste. I think using an indentation of 8 or more spaces will look better.","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"9fd7590c58fc632a2130d0ce3c5abbce5a1ed7a4","unresolved":false,"context_lines":[{"line_number":145,"context_line":"            return"},{"line_number":146,"context_line":"        lowername \u003d name.lower()"},{"line_number":147,"context_line":"        base_query \u003d model_query(models.Container).\\"},{"line_number":148,"context_line":"            filter(func.lower(models.Container.name) \u003d\u003d lowername)"},{"line_number":149,"context_line":"        if CONF.compute.unique_container_name_scope \u003d\u003d \u0027project\u0027:"},{"line_number":150,"context_line":"            container_with_same_name \u003d base_query.\\"},{"line_number":151,"context_line":"                filter_by(project_id\u003dcontext.project_id).\\"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_0a434743","line":148,"in_reply_to":"9a629dbe_7aabbed6","updated":"2016-11-09 04:09:59.000000000","message":"pep8 throws \u0027E127 continuation line over-indented for visual indent\u0027 if i use 8 space instead of 4 space.","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"dbbd03a908405222c085cb6ec08cf970912117e3","unresolved":false,"context_lines":[{"line_number":148,"context_line":"            filter(func.lower(models.Container.name) \u003d\u003d lowername)"},{"line_number":149,"context_line":"        if CONF.compute.unique_container_name_scope \u003d\u003d \u0027project\u0027:"},{"line_number":150,"context_line":"            container_with_same_name \u003d base_query.\\"},{"line_number":151,"context_line":"                filter_by(project_id\u003dcontext.project_id).\\"},{"line_number":152,"context_line":"                count()"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        elif CONF.compute.unique_container_name_scope \u003d\u003d \u0027global\u0027:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_fa850e59","line":151,"updated":"2016-11-08 20:08:41.000000000","message":"Ditto. Using 8 or more spaces might look better.","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"dbbd03a908405222c085cb6ec08cf970912117e3","unresolved":false,"context_lines":[{"line_number":150,"context_line":"            container_with_same_name \u003d base_query.\\"},{"line_number":151,"context_line":"                filter_by(project_id\u003dcontext.project_id).\\"},{"line_number":152,"context_line":"                count()"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        elif CONF.compute.unique_container_name_scope \u003d\u003d \u0027global\u0027:"},{"line_number":155,"context_line":"            container_with_same_name \u003d base_query.count()"},{"line_number":156,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_da268ab9","line":153,"updated":"2016-11-08 20:08:41.000000000","message":"nit: it is straight to have empty line between if branch and elif branch ...","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"9fd7590c58fc632a2130d0ce3c5abbce5a1ed7a4","unresolved":false,"context_lines":[{"line_number":150,"context_line":"            container_with_same_name \u003d base_query.\\"},{"line_number":151,"context_line":"                filter_by(project_id\u003dcontext.project_id).\\"},{"line_number":152,"context_line":"                count()"},{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        elif CONF.compute.unique_container_name_scope \u003d\u003d \u0027global\u0027:"},{"line_number":155,"context_line":"            container_with_same_name \u003d base_query.count()"},{"line_number":156,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_8a4e776d","line":153,"in_reply_to":"9a629dbe_da268ab9","updated":"2016-11-09 04:09:59.000000000","message":"Done","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"dbbd03a908405222c085cb6ec08cf970912117e3","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        elif CONF.compute.unique_container_name_scope \u003d\u003d \u0027global\u0027:"},{"line_number":155,"context_line":"            container_with_same_name \u003d base_query.count()"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        else:"},{"line_number":158,"context_line":"            return"},{"line_number":159,"context_line":"        if container_with_same_name \u003e 0:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_fa298e8a","line":156,"updated":"2016-11-08 20:08:41.000000000","message":"ditto","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"},{"author":{"_account_id":16277,"name":"Pradeep Kumar Singh","email":"ps4openstack@gmail.com","username":"pradeep_singh"},"change_message_id":"9fd7590c58fc632a2130d0ce3c5abbce5a1ed7a4","unresolved":false,"context_lines":[{"line_number":153,"context_line":""},{"line_number":154,"context_line":"        elif CONF.compute.unique_container_name_scope \u003d\u003d \u0027global\u0027:"},{"line_number":155,"context_line":"            container_with_same_name \u003d base_query.count()"},{"line_number":156,"context_line":""},{"line_number":157,"context_line":"        else:"},{"line_number":158,"context_line":"            return"},{"line_number":159,"context_line":"        if container_with_same_name \u003e 0:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a629dbe_aa517b10","line":156,"in_reply_to":"9a629dbe_fa298e8a","updated":"2016-11-09 04:09:59.000000000","message":"Done","commit_id":"ba6ecc986398a3ac620c14ba625d4aad8f6807c9"}]}