Recreate adapter only per block
This commit is contained in:
parent
f33ba13d74
commit
81c1207828
@ -1,3 +1,5 @@
|
|||||||
|
- 0.2.9
|
||||||
|
* Minimize instantiations of adapters in filter execution
|
||||||
- 0.2.8
|
- 0.2.8
|
||||||
* Upgrade chainsyncer
|
* Upgrade chainsyncer
|
||||||
- 0.2.7
|
- 0.2.7
|
||||||
|
@ -26,43 +26,69 @@ class StateFilter(SyncFilter):
|
|||||||
self.adapter_path = adapter_path
|
self.adapter_path = adapter_path
|
||||||
self.tx_adapter = tx_adapter
|
self.tx_adapter = tx_adapter
|
||||||
self.throttler = throttler
|
self.throttler = throttler
|
||||||
|
self.last_block_height = 0
|
||||||
|
self.adapter = None
|
||||||
|
self.store_lock = None
|
||||||
|
|
||||||
|
|
||||||
def filter(self, conn, block, tx, session=None):
|
def __get_adapter(self, block, force_reload=False):
|
||||||
cache_tx = None
|
if self.store_lock == None:
|
||||||
store_lock = StoreLock()
|
self.store_lock = StoreLock()
|
||||||
queue_adapter = None
|
|
||||||
|
reload = False
|
||||||
|
if block.number != self.last_block_height:
|
||||||
|
reload = True
|
||||||
|
elif self.adapter == None:
|
||||||
|
reload = True
|
||||||
|
elif force_reload:
|
||||||
|
reload = True
|
||||||
|
|
||||||
|
self.last_block_height = block.number
|
||||||
|
|
||||||
|
if reload:
|
||||||
while True:
|
while True:
|
||||||
|
logg.info('reloading adapter')
|
||||||
try:
|
try:
|
||||||
queue_adapter = ChaindFsAdapter(
|
self.adapter = ChaindFsAdapter(
|
||||||
self.chain_spec,
|
self.chain_spec,
|
||||||
self.adapter_path,
|
self.adapter_path,
|
||||||
self.tx_adapter,
|
self.tx_adapter,
|
||||||
None,
|
None,
|
||||||
)
|
)
|
||||||
|
break
|
||||||
except BackendError as e:
|
except BackendError as e:
|
||||||
logg.error('adapter instantiation failed: {}, one more try'.format(e))
|
logg.error('adapter instantiation failed: {}, one more try'.format(e))
|
||||||
store_lock.again()
|
self.store_lock.again()
|
||||||
continue
|
continue
|
||||||
|
|
||||||
store_lock.reset()
|
return self.adapter
|
||||||
|
|
||||||
|
|
||||||
|
def filter(self, conn, block, tx, session=None):
|
||||||
|
cache_tx = None
|
||||||
|
queue_adapter = self.__get_adapter(block)
|
||||||
|
|
||||||
|
self.store_lock.reset()
|
||||||
|
|
||||||
|
while True:
|
||||||
try:
|
try:
|
||||||
cache_tx = queue_adapter.get(tx.hash)
|
cache_tx = queue_adapter.get(tx.hash)
|
||||||
break
|
break
|
||||||
except NotLocalTxError:
|
except NotLocalTxError:
|
||||||
logg.debug('skipping not local transaction {}'.format(tx.hash))
|
logg.debug('skipping not local transaction {}'.format(tx.hash))
|
||||||
|
self.__stop_adapter()
|
||||||
return False
|
return False
|
||||||
except BackendError as e:
|
except BackendError as e:
|
||||||
logg.error('adapter get failed: {}, one more try'.format(e))
|
logg.error('adapter get failed: {}, one more try'.format(e))
|
||||||
queue_adapter = None
|
self.store_lock.again()
|
||||||
store_lock.again()
|
queue_adapter = self.__get_adapter(block, force_reload=True)
|
||||||
continue
|
continue
|
||||||
|
|
||||||
if cache_tx == None:
|
if cache_tx == None:
|
||||||
raise NotLocalTxError(tx.hash)
|
raise NotLocalTxError(tx.hash)
|
||||||
|
|
||||||
store_lock = StoreLock()
|
self.store_lock.reset()
|
||||||
|
|
||||||
queue_lock = StoreLock(error=QueueLockError)
|
queue_lock = StoreLock(error=QueueLockError)
|
||||||
while True:
|
while True:
|
||||||
try:
|
try:
|
||||||
@ -76,15 +102,18 @@ class StateFilter(SyncFilter):
|
|||||||
queue_lock.again()
|
queue_lock.again()
|
||||||
except FileNotFoundError as e:
|
except FileNotFoundError as e:
|
||||||
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
||||||
store_lock.again()
|
self.store_lock.again()
|
||||||
|
queue_adapter = self.__get_adapter(block, force_reload=True)
|
||||||
continue
|
continue
|
||||||
except NotLocalTxError as e:
|
except NotLocalTxError as e:
|
||||||
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
||||||
store_lock.again()
|
self.store_lock.again()
|
||||||
|
queue_adapter = self.__get_adapter(block, force_reload=True)
|
||||||
continue
|
continue
|
||||||
except StateLockedKey as e:
|
except StateLockedKey as e:
|
||||||
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
logg.debug('queue item {} not found, possible race condition, will retry: {}'.format(tx.hash, e))
|
||||||
store_lock.again()
|
self.store_lock.again()
|
||||||
|
queue_adapter = self.__get_adapter(block, force_reload=True)
|
||||||
continue
|
continue
|
||||||
|
|
||||||
logg.info('filter registered {} for {} in {}'.format(tx.status.name, tx.hash, block))
|
logg.info('filter registered {} for {} in {}'.format(tx.status.name, tx.hash, block))
|
||||||
|
@ -1,6 +1,6 @@
|
|||||||
[metadata]
|
[metadata]
|
||||||
name = chaind
|
name = chaind
|
||||||
version = 0.2.8
|
version = 0.2.9
|
||||||
description = Base package for chain queue service
|
description = Base package for chain queue service
|
||||||
author = Louis Holbrook
|
author = Louis Holbrook
|
||||||
author_email = dev@holbrook.no
|
author_email = dev@holbrook.no
|
||||||
|
Loading…
Reference in New Issue
Block a user