From 57845c07ed4e772abf4ddf353db8d08a2cca6fa0 Mon Sep 17 00:00:00 2001
From: Alan Jenkins <alan.christopher.jenkins@gmail.com>
Date: Mon, 10 Aug 2015 16:48:19 +0100
Subject: [PATCH] Clean up fds of segments we delete (during compaction)

When we delete a segment, let's close its fd as well.
Note as well wasting the fd, this was forcing the
filesystem to preserve the deleted file until we exited.

I noticed roughly 20 open fds of deleted files when
attic saved 10G of data.
---
 attic/repository.py | 3 +++
 1 file changed, 3 insertions(+)
 mode change 100644 => 100755 attic/repository.py

diff --git a/attic/repository.py b/attic/repository.py
old mode 100644
new mode 100755
index eed85dc43..1d2bb0f0f
--- a/attic/repository.py
+++ b/attic/repository.py
@@ -478,6 +478,9 @@ class LoggedIO(object):
             return fd
 
     def delete_segment(self, segment):
+        fd = self.fds.pop(segment)
+        if fd != None:
+            fd.close()
         try:
             os.unlink(self.segment_filename(segment))
         except OSError: