)]}'
{"quantum/api/v2/base.py":[{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"1497c2284d78528721e4a893c87b8baaa8e5c94e","unresolved":false,"context_lines":[{"line_number":463,"context_line":"                      if (\u0027required_by_policy\u0027 in value and"},{"line_number":464,"context_line":"                          (value[\u0027required_by_policy\u0027] or"},{"line_number":465,"context_line":"                           \u0027default\u0027 not in value))]"},{"line_number":466,"context_line":"        # The \u0027id\u0027 attribute should always be in field_list as it\u0027s needed"},{"line_number":467,"context_line":"        # for building some extensions\u0027 attributes"},{"line_number":468,"context_line":"        if \u0027id\u0027 not in field_list:"},{"line_number":469,"context_line":"            field_list.append(\u0027id\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAP3%2F%2F9SM%3D","line":466,"updated":"2013-07-08 15:38:25.000000000","message":"how about adding the field \u0027id\u0027 to the _attr_info data structure in order to avoid the lookup?","commit_id":"89202bed79729d5cb48c979d21c0ee991bad689a"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"5918cc1448b04b3b96fe53ea857082ccd97a21ca","unresolved":false,"context_lines":[{"line_number":463,"context_line":"                      if (\u0027required_by_policy\u0027 in value and"},{"line_number":464,"context_line":"                          (value[\u0027required_by_policy\u0027] or"},{"line_number":465,"context_line":"                           \u0027default\u0027 not in value))]"},{"line_number":466,"context_line":"        # The \u0027id\u0027 attribute should always be in field_list as it\u0027s needed"},{"line_number":467,"context_line":"        # for building some extensions\u0027 attributes"},{"line_number":468,"context_line":"        if \u0027id\u0027 not in field_list:"},{"line_number":469,"context_line":"            field_list.append(\u0027id\u0027)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAP3%2F%2Fjrw%3D","line":466,"in_reply_to":"AAAAP3%2F%2F9SM%3D","updated":"2013-07-16 15:54:12.000000000","message":"it is there, but it would be filtered out because it usually declared as not required_by_policy.\nLet me see if adding primary_key to the conditions for being in field_list would work.","commit_id":"89202bed79729d5cb48c979d21c0ee991bad689a"}]}