diff options
| author | Prashanth Pai <ppai@redhat.com> | 2016-03-18 18:09:46 +0530 | 
|---|---|---|
| committer | Thiago da Silva <thiago@redhat.com> | 2016-04-15 05:27:54 -0700 | 
| commit | 2bd696e392e420a2521dcca0b8613122d8169025 (patch) | |
| tree | 9cd42e35957c0df07696e160f357019fde78c2d4 /test/unit/common | |
| parent | 43da7927560811c55838a6a1c2d0ee1a52aada40 (diff) | |
Enhance object expiration
This change re-introduces object expiration feature with some internal
differences from earlier implementation such as:
* Creation of zero-byte tracker object is performed directly on the
  mount point. Earlier HTTP request was sent to object server
  for the same. This incurred overhead of metadata creation for
  these zero-byte files which isn't necessarry as all required
  information is encoded in the path itself.
* Crawling of zero-byte tracker objects is done by the object
  expirer daemon itself and not container server.
* Deletion of tracker object is performed by the object expiration daemon
  directly on mount point.
Deletion of actual data object is not carried out by object expiration
daemon directly. The object expirer sends a DELETE request to object
server which deletes the actual object. This behaviour is not changed.
There is no change in behaviour in comparison with older implementation.
This is asserted by re-enabling existing functional tests without
any changes.
Change-Id: I01dc77cc4db3be3147d54e3aa2a19ed182498900
Signed-off-by: Prashanth Pai <ppai@redhat.com>
Reviewed-on: http://review.gluster.org/13913
Reviewed-by: Thiago da Silva <thiago@redhat.com>
Tested-by: Thiago da Silva <thiago@redhat.com>
Diffstat (limited to 'test/unit/common')
| -rw-r--r-- | test/unit/common/test_diskdir.py | 2 | 
1 files changed, 1 insertions, 1 deletions
diff --git a/test/unit/common/test_diskdir.py b/test/unit/common/test_diskdir.py index 3c91016..623164c 100644 --- a/test/unit/common/test_diskdir.py +++ b/test/unit/common/test_diskdir.py @@ -461,7 +461,7 @@ class TestContainerBroker(unittest.TestCase):          self.assert_(broker.empty())          obj = self._create_file('o.txt')          self.assert_(not broker.empty()) -        broker.delete_object('o', normalize_timestamp(time())) +        broker.delete_object('o', normalize_timestamp(time()), 0)          # delete_object() should be a NOOP          self.assert_(not broker.empty())          os.unlink(obj)  | 
