)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"b9f9ef557b5fa94c072ebdedc81ac3e17232abf6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     WenjunWang1992 \u003c10191230@zte.com.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-04 10:25:19 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Optimize service find in migration"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"A little change of code when find source volume host, make the"},{"line_number":10,"context_line":"code more pythonic."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7aa08908_bd2beb32","line":7,"updated":"2016-06-04 06:13:41.000000000","message":"-1: I don\u0027t think there is any optimization here; it only uses a python construct which is slightly confusing. Please fix the commit summary.","commit_id":"ab50bb9db3589362f422820ee55e86f685ce0926"}],"cinder/volume/api.py":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"b9f9ef557b5fa94c072ebdedc81ac3e17232abf6","unresolved":false,"context_lines":[{"line_number":1301,"context_line":"        for service in services:"},{"line_number":1302,"context_line":"            if utils.service_is_up(service) and service.host \u003d\u003d svc_host:"},{"line_number":1303,"context_line":"                break"},{"line_number":1304,"context_line":"        else:"},{"line_number":1305,"context_line":"            msg \u003d _(\u0027No available service named %s\u0027) % host"},{"line_number":1306,"context_line":"            LOG.error(msg)"},{"line_number":1307,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_3df9bbc9","line":1304,"updated":"2016-06-04 06:13:41.000000000","message":"I find the existing code more readable.","commit_id":"ab50bb9db3589362f422820ee55e86f685ce0926"},{"author":{"_account_id":19919,"name":"Wenjun Wang","email":"wenjun_dev@126.com","username":"June"},"change_message_id":"21d72e472cd48d99d91203b2122e287cccf1105c","unresolved":false,"context_lines":[{"line_number":1301,"context_line":"        for service in services:"},{"line_number":1302,"context_line":"            if utils.service_is_up(service) and service.host \u003d\u003d svc_host:"},{"line_number":1303,"context_line":"                break"},{"line_number":1304,"context_line":"        else:"},{"line_number":1305,"context_line":"            msg \u003d _(\u0027No available service named %s\u0027) % host"},{"line_number":1306,"context_line":"            LOG.error(msg)"},{"line_number":1307,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_7d76a33f","line":1304,"in_reply_to":"7aa08908_3df9bbc9","updated":"2016-06-04 06:25:44.000000000","message":"yes, it is, just code refactor, I will abandon if most of reviewers agree with you, thanks for review.","commit_id":"ab50bb9db3589362f422820ee55e86f685ce0926"},{"author":{"_account_id":18026,"name":"huanan","email":"huanan@huawei.com","username":"huanan"},"change_message_id":"94c4241e6602b00261e0d3eca624cba2a838b161","unresolved":false,"context_lines":[{"line_number":1301,"context_line":"        for service in services:"},{"line_number":1302,"context_line":"            if utils.service_is_up(service) and service.host \u003d\u003d svc_host:"},{"line_number":1303,"context_line":"                break"},{"line_number":1304,"context_line":"        else:"},{"line_number":1305,"context_line":"            msg \u003d _(\u0027No available service named %s\u0027) % host"},{"line_number":1306,"context_line":"            LOG.error(msg)"},{"line_number":1307,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_ba157f4a","line":1304,"in_reply_to":"7aa08908_7d76a33f","updated":"2016-06-16 04:02:32.000000000","message":"I think so, original code is better.","commit_id":"ab50bb9db3589362f422820ee55e86f685ce0926"}]}