)]}'
{"cinder/api/validation/validators.py":[{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"fadca9e0fda1e5742311451082dcfaf31cfec09b","unresolved":false,"context_lines":[{"line_number":222,"context_line":"    bad_keys \u003d []"},{"line_number":223,"context_line":"    for key, value in quota_set.items():"},{"line_number":224,"context_line":"        if (key not in QUOTAS and key not in GROUP_QUOTAS and key not in"},{"line_number":225,"context_line":"                NON_QUOTA_KEYS):"},{"line_number":226,"context_line":"            bad_keys.append(key)"},{"line_number":227,"context_line":"            continue"},{"line_number":228,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_b8134c5c","line":225,"range":{"start_line":225,"start_character":15,"end_line":225,"end_character":32},"updated":"2018-04-19 11:52:22.000000000","message":"NON_QUOTA_KEYS will never be used in the business logic, is it possible to remove them here?","commit_id":"663c00b586514da52b954db8edd276efaeb5bf4b"},{"author":{"_account_id":24645,"name":"Pooja Jadhav","email":"pooja.jadhav@nttdata.com","username":"poojajadhav"},"change_message_id":"d84d298bf1422dbf5efa7057bf7c3f19ec8b53a7","unresolved":false,"context_lines":[{"line_number":222,"context_line":"    bad_keys \u003d []"},{"line_number":223,"context_line":"    for key, value in quota_set.items():"},{"line_number":224,"context_line":"        if (key not in QUOTAS and key not in GROUP_QUOTAS and key not in"},{"line_number":225,"context_line":"                NON_QUOTA_KEYS):"},{"line_number":226,"context_line":"            bad_keys.append(key)"},{"line_number":227,"context_line":"            continue"},{"line_number":228,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_b0975345","line":225,"range":{"start_line":225,"start_character":15,"end_line":225,"end_character":32},"in_reply_to":"9f6a8fd7_b8134c5c","updated":"2018-04-26 08:49:36.000000000","message":"I agree with your suggestion, but it will need lot of refactoring so IMO, I will create separate patch for this change.","commit_id":"663c00b586514da52b954db8edd276efaeb5bf4b"}]}