backup/warcprox
1
0
Fork 0
mirror of https://github.com/internetarchive/warcprox.git synced 2025-01-18 13:22:09 +01:00
Code Issues Projects Releases Activity

standardize the concept of 'now' to ensure that the same view of the service is returned from the read and update queries.

Browse Source
This commit is contained in:
James Kafader 2017-05-16 11:31:03 -07:00
parent 501ec1ff23
commit 78c26186b0
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
doublethink/services.py
View File

@ -222,6 +222,8 @@ class ServiceRegistry(object):
'''
if candidate is not None:
candidate['id'] = role
# use the same concept of 'now' for both queries
now = rr.now().run()
if not 'heartbeat_interval' in candidate:
raise Exception(
@ -231,8 +233,8 @@ class ServiceRegistry(object):
if not (isinstance(val, float) or isinstance(val, int)) or val <= 0:
raise Exception('heartbeat_interval must be a number > 0')
candidate['first_heartbeat'] = r.now()
candidate['last_heartbeat'] = r.now()
candidate['first_heartbeat'] = now
candidate['last_heartbeat'] = now
if not 'host' in candidate:
candidate['host'] = socket.gethostname()
if not 'pid' in candidate:
@ -243,7 +245,7 @@ class ServiceRegistry(object):
lambda row: r.branch(
r.branch(
row,
row['last_heartbeat'] > r.now() - row['heartbeat_interval'] * 3,
row['last_heartbeat'] > now - row['heartbeat_interval'] * 3,
False),
row, candidate),
return_changes='always').run()
@ -256,7 +258,7 @@ class ServiceRegistry(object):
results = list(self.rr.table(
'services', read_mode='majority').get_all(role).filter(
lambda row: row['last_heartbeat'] > r.now() - row['heartbeat_interval'] * 3).run())
lambda row: row['last_heartbeat'] > now - row['heartbeat_interval'] * 3).run())
if results:
return results[0]
else: