)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"3ce9afad02175e429e3566a4ae2999668795720a","unresolved":false,"context_lines":[{"line_number":3249,"context_line":"                        filter_by(id\u003dsecurity_group_rule_id).\\"},{"line_number":3250,"context_line":"                        soft_delete()"},{"line_number":3251,"context_line":"        if count \u003d\u003d 0:"},{"line_number":3252,"context_line":"            raise exception.SecurityGroupNotFoundForRule("},{"line_number":3253,"context_line":"                                               rule_id\u003dsecurity_group_rule_id)"},{"line_number":3254,"context_line":""},{"line_number":3255,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F4Xk%3D","line":3252,"updated":"2012-12-27 01:34:03.000000000","message":"As it appears this exception is new to this function, a unit test for this would be nice","commit_id":"5425824271b36c3b9822c76af47c27fc3f338df5"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"146a8185073e1ee15fde02704e5dbabd80b1d597","unresolved":false,"context_lines":[{"line_number":3249,"context_line":"                        filter_by(id\u003dsecurity_group_rule_id).\\"},{"line_number":3250,"context_line":"                        soft_delete()"},{"line_number":3251,"context_line":"        if count \u003d\u003d 0:"},{"line_number":3252,"context_line":"            raise exception.SecurityGroupNotFoundForRule("},{"line_number":3253,"context_line":"                                               rule_id\u003dsecurity_group_rule_id)"},{"line_number":3254,"context_line":""},{"line_number":3255,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2F4Vs%3D","line":3252,"in_reply_to":"AAAAM3%2F%2F4Xk%3D","updated":"2012-12-27 05:07:17.000000000","message":"This exception is not new for this method. Take a look at:\ndef security_group_rule_get:\n    result \u003d _security_group_get_query(context, session\u003dsession,\n                                       project_only\u003dTrue).\\\n                    filter_by(id\u003dsecurity_group_id).\\\n                    options(joinedload_all(\u0027instances\u0027)).\\\n                    first()\n    if not result:\n        raise exception.SecurityGroupNotFound(\n                security_group_id\u003dsecurity_group_id)\n    return result\n\nSo the old and new methods have the same behavior.","commit_id":"5425824271b36c3b9822c76af47c27fc3f338df5"}]}