)]}'
{"nova/db/sqlalchemy/api.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"63803187e4f565823798c6e49c3f9734ac5659d3","unresolved":false,"context_lines":[{"line_number":726,"context_line":"            addr \u003d ip[\u0027address\u0027]"},{"line_number":727,"context_line":"            if (addr in existing_ips and"},{"line_number":728,"context_line":"                ip.get(\u0027id\u0027) !\u003d existing_ips[addr][\u0027id\u0027]):"},{"line_number":729,"context_line":"                raise exception.FloatingIpExists(**dict(existing_ips[addr]))"},{"line_number":730,"context_line":""},{"line_number":731,"context_line":"            model \u003d models.FloatingIp()"},{"line_number":732,"context_line":"            model.update(ip)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAALH%2F%2FiMY%3D","line":729,"updated":"2012-09-03 16:19:38.000000000","message":"Would it make sense to do the existence check in a separate loop before begin()ing the session, so as to fail fast if any of the ips are preexisting?","commit_id":"94c74572e724c8beddf6dcc4547443c5607a290c"}]}