Add close database for dump/restore since we keep cursors

This commit is contained in:
Cédric Krier
2009-05-03 14:34:47 +02:00
parent a8ca5195f9
commit edf78613bf
2 changed files with 7 additions and 1 deletions
+2
View File
@@ -140,8 +140,10 @@ class Database(DatabaseInterface):
cursor = database.cursor()
if not cursor.test():
cursor.close()
database.close()
raise Exception('Couldn\'t restore database!')
cursor.close()
database.close()
return True
@staticmethod
+5 -1
View File
@@ -237,8 +237,12 @@ def drop(database_name, password):
return True
def dump(database_name, password):
logger = logging.getLogger('database')
security.check_super(password)
Database(database_name).close()
# Sleep to let connections close
time.sleep(1)
logger = logging.getLogger('database')
data = Database.dump(database_name)
logger.info('DUMP DB: %s' % (database_name))
return base64.encodestring(data)