)]}'
{"cinder/volume/drivers/dell/dell_storagecenter_common.py":[{"author":{"_account_id":2759,"name":"Huang Zhiteng","email":"winston.d@gmail.com","username":"zhiteng-huang"},"change_message_id":"86ee762ed07653f2f62b9af4e3f21f9e884cf559","unresolved":false,"context_lines":[{"line_number":1446,"context_line":"                    volume_updates.append({\u0027volume_id\u0027: volume[\u0027id\u0027],"},{"line_number":1447,"context_line":"                                           \u0027updates\u0027: model_update})"},{"line_number":1448,"context_line":"                # Let\u0027s do up to 5 replications at once."},{"line_number":1449,"context_line":"                if len(replitems) \u003d\u003d 5:"},{"line_number":1450,"context_line":"                    volume_updates +\u003d self._finish_failback(api, replitems)"},{"line_number":1451,"context_line":"                    replitems \u003d []"},{"line_number":1452,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_750b277e","line":1449,"updated":"2016-08-25 00:28:29.000000000","message":"I guess it is not unlikely for deployer to change this concurrency to some other value for a deployment, but it\u0027s always better to at least use a CONST (if not a configurable value) rather than hardcoded integer.","commit_id":"2a20f1f59036a0e9d4f23a9282a4b9a712b493f5"},{"author":{"_account_id":12112,"name":"Tom Swanson","email":"tom_swanson@dell.com","username":"tomswanson"},"change_message_id":"20a8868c4f231af559f2ee17686a86fe042ad183","unresolved":false,"context_lines":[{"line_number":1446,"context_line":"                    volume_updates.append({\u0027volume_id\u0027: volume[\u0027id\u0027],"},{"line_number":1447,"context_line":"                                           \u0027updates\u0027: model_update})"},{"line_number":1448,"context_line":"                # Let\u0027s do up to 5 replications at once."},{"line_number":1449,"context_line":"                if len(replitems) \u003d\u003d 5:"},{"line_number":1450,"context_line":"                    volume_updates +\u003d self._finish_failback(api, replitems)"},{"line_number":1451,"context_line":"                    replitems \u003d []"},{"line_number":1452,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1ac06dbe_ddbf62fa","line":1449,"in_reply_to":"1ac06dbe_750b277e","updated":"2016-08-25 19:25:18.000000000","message":"Good point. If I have to push another patch I will change this.","commit_id":"2a20f1f59036a0e9d4f23a9282a4b9a712b493f5"}]}