)]}'
{"designate/central/service.py":[{"author":{"_account_id":8094,"name":"Vinod Mangalpally","email":"mrvinod@outlook.com","username":"vinodmr"},"change_message_id":"f020a01eec9b99205d3fe10631d8e9fc0b3ffa16","unresolved":false,"context_lines":[{"line_number":772,"context_line":"        results \u003d {}"},{"line_number":773,"context_line":""},{"line_number":774,"context_line":"        for domain in domains:"},{"line_number":775,"context_line":"            servers \u003d self.storage_api.find_servers(context)"},{"line_number":776,"context_line":"            records \u003d self.storage_api.find_records(context, domain[\u0027id\u0027])"},{"line_number":777,"context_line":""},{"line_number":778,"context_line":"            with wrap_backend_call():"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Bm0w%3D","line":775,"updated":"2013-11-22 15:21:18.000000000","message":"Should this be changed too or does domain[\u0027id] transform to a criterion automatically?","commit_id":"8c41cac0610aad4923169f2e4556e3040211b14a"},{"author":{"_account_id":8094,"name":"Vinod Mangalpally","email":"mrvinod@outlook.com","username":"vinodmr"},"change_message_id":"f020a01eec9b99205d3fe10631d8e9fc0b3ffa16","unresolved":false,"context_lines":[{"line_number":793,"context_line":"        }"},{"line_number":794,"context_line":""},{"line_number":795,"context_line":"        policy.check(\u0027diagnostics_sync_domain\u0027, context, target)"},{"line_number":796,"context_line":""},{"line_number":797,"context_line":"        records \u003d self.storage_api.find_records(context, domain_id)"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"        with wrap_backend_call():"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2Bmz4%3D","line":796,"updated":"2013-11-22 15:21:18.000000000","message":"Likewise here - is a change needed here?","commit_id":"8c41cac0610aad4923169f2e4556e3040211b14a"},{"author":{"_account_id":8094,"name":"Vinod Mangalpally","email":"mrvinod@outlook.com","username":"vinodmr"},"change_message_id":"7253b565c3b6257aaea6e98877b4995acf5da6ae","unresolved":false,"context_lines":[{"line_number":797,"context_line":"        policy.check(\u0027diagnostics_sync_domain\u0027, context, target)"},{"line_number":798,"context_line":""},{"line_number":799,"context_line":"        records \u003d self.storage_api.find_records("},{"line_number":800,"context_line":"            context, criterion\u003d{\u0027domain_id\u0027: domain_id})"},{"line_number":801,"context_line":""},{"line_number":802,"context_line":"        with wrap_backend_call():"},{"line_number":803,"context_line":"            return self.backend.sync_domain(context, domain, records)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAATn%2F%2BmIg%3D","line":800,"updated":"2013-11-22 17:13:20.000000000","message":"minor nitpick on style - sync_domains has criterion defined outside the call to find_records, this has in the call.  No need to change unless you are making another change","commit_id":"741441cfdf6cd25c99017dabf858f6082304cbf3"}]}