)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"33692542ff921e1d30e8f43dc2dff4de601d5507","unresolved":false,"context_lines":[{"line_number":4267,"context_line":"@require_context"},{"line_number":4268,"context_line":"@main_context_manager.reader"},{"line_number":4269,"context_line":"def security_group_get(context, security_group_id, columns_to_join\u003dNone,"},{"line_number":4270,"context_line":"                       join_rules\u003dTrue):"},{"line_number":4271,"context_line":"    query \u003d _security_group_get_query(context, project_only\u003dTrue,"},{"line_number":4272,"context_line":"                                      join_rules\u003djoin_rules).\\"},{"line_number":4273,"context_line":"                    filter_by(id\u003dsecurity_group_id)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9ad45d7e_c10c6f3d","line":4270,"range":{"start_line":4270,"start_character":23,"end_line":4270,"end_character":38},"updated":"2016-08-10 17:55:20.000000000","message":"Can\u0027t we do away with this and instead set it to:\n\n join_rules \u003d \u0027rules\u0027 in columns_to_join\n if join_rules:\n     columns_to_join.remove(\u0027rules\u0027)\n\n? That would keep the interface much more consistent with how we do this elsewhere.","commit_id":"2c4aae2b2a25316d57b0be9e9c01eb932d0dfc88"}]}