)]}'
{"karbor/tests/fullstack/test_scheduled_operations.py":[{"author":{"_account_id":18266,"name":"zengchen","email":"chenzeng2@huawei.com","username":"zengchen"},"change_message_id":"5ced3deddaa64f3c8146a6f32503786aed8c91cf","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        operation_items \u003d self.karbor_client.scheduled_operations.list()"},{"line_number":30,"context_line":"        before_num \u003d len(operation_items)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        volume \u003d self.store(objects.Volume())"},{"line_number":33,"context_line":"        volume.create(1)"},{"line_number":34,"context_line":"        plan \u003d self.store(objects.Plan())"},{"line_number":35,"context_line":"        plan.create(self.provider_id, [volume, ])"},{"line_number":36,"context_line":"        trigger \u003d self.store(objects.Trigger())"},{"line_number":37,"context_line":"        trigger.create(\u0027time\u0027, {\u0027pattern\u0027: \u00270 20 * * 2\u0027, \u0027format\u0027: \u0027crontab\u0027})"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        operation \u003d self.store(objects.ScheduledOperation())"},{"line_number":40,"context_line":"        operation.create("},{"line_number":41,"context_line":"            \u0027protect\u0027,"},{"line_number":42,"context_line":"            trigger.id,"},{"line_number":43,"context_line":"            {\u0027plan_id\u0027: plan.id, \u0027provider_id\u0027: self.provider_id}"},{"line_number":44,"context_line":"        )"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        operation_items \u003d self.karbor_client.scheduled_operations.list()"},{"line_number":47,"context_line":"        after_num \u003d len(operation_items)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a8ec9b2_04225301","line":44,"range":{"start_line":32,"start_character":0,"end_line":44,"end_character":9},"updated":"2016-09-21 02:08:24.000000000","message":"These codes are repeated in all functions, could you please move them to one method, and reuse it? Thanks.","commit_id":"99bcbda91169bf9db2747732ea05743b3debbe5c"},{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"dd069fb382210d0a68a8c2a37485f68f5f1fcc30","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        operation_items \u003d self.karbor_client.scheduled_operations.list()"},{"line_number":30,"context_line":"        before_num \u003d len(operation_items)"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"        volume \u003d self.store(objects.Volume())"},{"line_number":33,"context_line":"        volume.create(1)"},{"line_number":34,"context_line":"        plan \u003d self.store(objects.Plan())"},{"line_number":35,"context_line":"        plan.create(self.provider_id, [volume, ])"},{"line_number":36,"context_line":"        trigger \u003d self.store(objects.Trigger())"},{"line_number":37,"context_line":"        trigger.create(\u0027time\u0027, {\u0027pattern\u0027: \u00270 20 * * 2\u0027, \u0027format\u0027: \u0027crontab\u0027})"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"        operation \u003d self.store(objects.ScheduledOperation())"},{"line_number":40,"context_line":"        operation.create("},{"line_number":41,"context_line":"            \u0027protect\u0027,"},{"line_number":42,"context_line":"            trigger.id,"},{"line_number":43,"context_line":"            {\u0027plan_id\u0027: plan.id, \u0027provider_id\u0027: self.provider_id}"},{"line_number":44,"context_line":"        )"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"        operation_items \u003d self.karbor_client.scheduled_operations.list()"},{"line_number":47,"context_line":"        after_num \u003d len(operation_items)"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a8ec9b2_773aeb99","line":44,"range":{"start_line":32,"start_character":0,"end_line":44,"end_character":9},"in_reply_to":"7a8ec9b2_04225301","updated":"2016-09-21 02:31:37.000000000","message":"ok","commit_id":"99bcbda91169bf9db2747732ea05743b3debbe5c"},{"author":{"_account_id":18266,"name":"zengchen","email":"chenzeng2@huawei.com","username":"zengchen"},"change_message_id":"851c3602f2af5b395466032484527a7379faa067","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        volume.create(1)"},{"line_number":33,"context_line":"        plan \u003d self.store(objects.Plan())"},{"line_number":34,"context_line":"        plan.create(self.provider_id, [volume, ])"},{"line_number":35,"context_line":"        trigger \u003d self.store(objects.Trigger())"},{"line_number":36,"context_line":"        trigger.create(\u0027time\u0027, trigger_properties)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        operation \u003d objects.ScheduledOperation()"},{"line_number":39,"context_line":"        operation.create("},{"line_number":40,"context_line":"            \u0027protect\u0027,"},{"line_number":41,"context_line":"            trigger.id,"},{"line_number":42,"context_line":"            {\u0027plan_id\u0027: plan.id, \u0027provider_id\u0027: self.provider_id},"},{"line_number":43,"context_line":"            name"},{"line_number":44,"context_line":"        )"},{"line_number":45,"context_line":"        return operation"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def _create_scheduled_operation_for_server(self,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a8bc5a2_868df07d","line":44,"range":{"start_line":35,"start_character":0,"end_line":44,"end_character":9},"updated":"2016-09-22 06:25:08.000000000","message":"These codes are repeated. Define a function named \u0027_create_scheduled_operation(trigger_properties, operation_name, operation_definition)\u0027, and move them to it. How about it?","commit_id":"75d0c568bfcdb82b00d266a06ce3df08ce5a31e8"},{"author":{"_account_id":21648,"name":"zhangshuai","email":"446077695@qq.com","username":"zhangshuai"},"change_message_id":"f5e94ec9fb550eabe3baf89ca467ae470518fb04","unresolved":false,"context_lines":[{"line_number":32,"context_line":"        volume.create(1)"},{"line_number":33,"context_line":"        plan \u003d self.store(objects.Plan())"},{"line_number":34,"context_line":"        plan.create(self.provider_id, [volume, ])"},{"line_number":35,"context_line":"        trigger \u003d self.store(objects.Trigger())"},{"line_number":36,"context_line":"        trigger.create(\u0027time\u0027, trigger_properties)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"        operation \u003d objects.ScheduledOperation()"},{"line_number":39,"context_line":"        operation.create("},{"line_number":40,"context_line":"            \u0027protect\u0027,"},{"line_number":41,"context_line":"            trigger.id,"},{"line_number":42,"context_line":"            {\u0027plan_id\u0027: plan.id, \u0027provider_id\u0027: self.provider_id},"},{"line_number":43,"context_line":"            name"},{"line_number":44,"context_line":"        )"},{"line_number":45,"context_line":"        return operation"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"    def _create_scheduled_operation_for_server(self,"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a8bc5a2_31c45891","line":44,"range":{"start_line":35,"start_character":0,"end_line":44,"end_character":9},"in_reply_to":"5a8bc5a2_868df07d","updated":"2016-09-22 06:46:39.000000000","message":"ok","commit_id":"75d0c568bfcdb82b00d266a06ce3df08ce5a31e8"}]}