)]}'
{"nova/openstack/common/db/common.py":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"357f4818bcdeb586527388112b44f618f0bcb181","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        super(DBError, self).__init__(str(inner_exception))"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"def wrap_db_error(f):"},{"line_number":45,"context_line":"    def _wrap(*args, **kwargs):"},{"line_number":46,"context_line":"        try:"},{"line_number":47,"context_line":"            return f(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Fgvk%3D","line":44,"updated":"2013-01-17 20:55:36.000000000","message":"You should also add DBDuplicateEntry exception from this patch:\nhttps://review.openstack.org/#/c/16339/7/nova/db/sqlalchemy/session.py\n\nIt is very important for bp/db-unique-keys implementation.","commit_id":"924c5f7d3fda209ec10d037985f5a1f40ff1a126"},{"author":{"_account_id":159,"name":"Eric Windisch","email":"ewindisch@docker.com","username":"ewindisch"},"change_message_id":"7be97c0122ce927d9a5676158f3705daaea58d65","unresolved":false,"context_lines":[{"line_number":41,"context_line":"        super(DBError, self).__init__(str(inner_exception))"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"def wrap_db_error(f):"},{"line_number":45,"context_line":"    def _wrap(*args, **kwargs):"},{"line_number":46,"context_line":"        try:"},{"line_number":47,"context_line":"            return f(*args, **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2FguE%3D","line":44,"in_reply_to":"AAAAM3%2F%2Fgvk%3D","updated":"2013-01-17 21:09:38.000000000","message":"The code actually uses the version of this method that exists in session.py. I forgot to delete the method from here.","commit_id":"924c5f7d3fda209ec10d037985f5a1f40ff1a126"}],"nova/openstack/common/db/sqlalchemy/session.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"94c5052a3572d65e69e64b83e5b0b3c5312bd333","unresolved":false,"context_lines":[{"line_number":268,"context_line":"    cfg.StrOpt(\u0027sql_connection\u0027,"},{"line_number":269,"context_line":"               default\u003d\u0027sqlite:///\u0027 +"},{"line_number":270,"context_line":"                       os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":271,"context_line":"                       \u0027../\u0027, \u0027$sqlite_db\u0027)),"},{"line_number":272,"context_line":"               help\u003d\u0027The SQLAlchemy connection string used to connect to the \u0027"},{"line_number":273,"context_line":"                    \u0027database\u0027),"},{"line_number":274,"context_line":"    cfg.StrOpt(\u0027sqlite_db\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FJn8%3D","line":271,"updated":"2013-02-06 11:25:44.000000000","message":"is this really the default path you want? this seems complicated enough to have a comment about it.","commit_id":"cd0c765ced2823c05b16126d6ba6a823bb66fe7d"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"aa0729c307217f64ba36dd66c684d38a9bb78186","unresolved":false,"context_lines":[{"line_number":268,"context_line":"    cfg.StrOpt(\u0027sql_connection\u0027,"},{"line_number":269,"context_line":"               default\u003d\u0027sqlite:///\u0027 +"},{"line_number":270,"context_line":"                       os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":271,"context_line":"                       \u0027../\u0027, \u0027$sqlite_db\u0027)),"},{"line_number":272,"context_line":"               help\u003d\u0027The SQLAlchemy connection string used to connect to the \u0027"},{"line_number":273,"context_line":"                    \u0027database\u0027),"},{"line_number":274,"context_line":"    cfg.StrOpt(\u0027sqlite_db\u0027,"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2FH64%3D","line":271,"in_reply_to":"AAAAM3%2F%2FJn8%3D","updated":"2013-02-06 14:31:37.000000000","message":"We override the default in nova.config using set_defaults()","commit_id":"cd0c765ced2823c05b16126d6ba6a823bb66fe7d"}]}
