Skip to content

Commit a63998d

Browse files
cleanup method for resolver thread
1 parent 73170cc commit a63998d

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

mongodb_consistent_backup/Oplog/Resolver/ResolverThread.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,13 @@ def __init__(self, state, uri, tailed_oplog, mongodump_oplog, max_end_ts, dump_g
2121
self.oplogs = {}
2222
self.changes = 0
2323

24+
def cleanup(self):
25+
if 'tailed' in self.oplogs:
26+
self.oplogs['tailed'].close()
27+
del self.oplogs['tailed']
28+
if 'file' in self.tailed_opplog and os.path.isfile(self.tailed_oplog['file']):
29+
os.remove(self.tailed_oplog['file'])
30+
2431
def run(self):
2532
self.oplogs['backup'] = Oplog(self.mongodump_oplog['file'], self.dump_gzip, 'a+')
2633
self.oplogs['tailed'] = Oplog(self.tailed_oplog['file'], self.dump_gzip)
@@ -40,11 +47,6 @@ def run(self):
4047
elif self.last_ts > self.max_end_ts:
4148
break
4249

43-
# remove temporary tailed oplog
44-
self.oplogs['tailed'].close()
45-
os.remove(self.tailed_oplog['file'])
46-
del self.oplogs['tailed']
47-
4850
self.state.set('count', self.mongodump_oplog['count'] + self.changes)
4951
self.state.set('last_ts', self.last_ts)
5052
self.state.set('running', False)
@@ -59,6 +61,7 @@ def run(self):
5961
sys.exit(self.exit_code)
6062

6163
def close(self):
64+
self.cleanup()
6265
if len(self.oplogs) > 0:
6366
for oplog in self.oplogs:
6467
self.oplogs[oplog].flush()

0 commit comments

Comments
 (0)