Commit 88fd7df2 authored by Chrysostomos Nanakos's avatar Chrysostomos Nanakos
Browse files

pithos: Change exception type

Raise IllegalOperationError instead of NotAllowedError when
consequent map checks are requested before the map check interval
parent 6feb2c27
......@@ -979,7 +979,7 @@ class ModularBackend(BaseBackend):
if not props[self.AVAILABLE]:
except (NotAllowedError, IllegalOperationError):
except IllegalOperationError:
pass # just update the database
# get updated properties
......@@ -1144,8 +1144,8 @@ class ModularBackend(BaseBackend):
if props[self.MAP_CHECK_TIMESTAMP]:
elapsed_time = time() - float(props[self.MAP_CHECK_TIMESTAMP])
if elapsed_time < self.map_check_interval:
raise NotAllowedError(
'Consequent map checks are limited: retry later.')
raise IllegalOperationError(
'Unable to retrieve Archipelago volume hashmap')
hashmap =[self.HASH], props[self.SIZE])
except: # map does not exist
......@@ -1158,7 +1158,7 @@ class ModularBackend(BaseBackend):
self.wrapper.commit() # commit transaction
self.wrapper.execute() # start new transaction
raise IllegalOperationError(
'Unable to retrieve Archipelago Volume hashmap.')
'Unable to retrieve Archipelago volume hashmap')
else: # map exists
'available', True)
......@@ -1339,7 +1339,7 @@ class ModularBackend(BaseBackend):
if props[self.IS_SNAPSHOT]:
raise IllegalOperationError(
'Cannot update Archipelago Volume hashmap.')
'Cannot update Archipelago volume hashmap.')
meta = meta or {}
if size == 0: # No such thing as an empty hashmap.
hashmap = [self.put_block('')]
......@@ -1784,7 +1784,7 @@ class ModularBackend(BaseBackend):
is_snapshot, offset)
if is_snapshot:
raise IllegalOperationError(
'Cannot update an Archipelago Volume block.')
'Cannot update an Archipelago volume block.')
if offset == 0 and len(data) == self.block_size:
return self.put_block(data)
h =, offset, data)
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment