Renew backend every filter pass, allow race in backend
This commit is contained in:
parent
96bdca20cc
commit
4be5325df2
@ -10,6 +10,7 @@ from chainqueue.store.fs import (
|
|||||||
IndexStore,
|
IndexStore,
|
||||||
CounterStore,
|
CounterStore,
|
||||||
)
|
)
|
||||||
|
from chainqueue.error import BackendIntegrityError
|
||||||
from shep.store.file import SimpleFileStoreFactory
|
from shep.store.file import SimpleFileStoreFactory
|
||||||
from shep.error import StateInvalid
|
from shep.error import StateInvalid
|
||||||
|
|
||||||
@ -42,16 +43,24 @@ class ChaindFsAdapter(ChaindAdapter):
|
|||||||
except StateInvalid as e:
|
except StateInvalid as e:
|
||||||
logg.error('I am just a simple syncer and do not know how to handle the state which the tx {} is in: {}'.format(tx_hash, e))
|
logg.error('I am just a simple syncer and do not know how to handle the state which the tx {} is in: {}'.format(tx_hash, e))
|
||||||
return None
|
return None
|
||||||
|
except FileNotFoundError:
|
||||||
|
pass
|
||||||
|
if v ==None:
|
||||||
|
raise BackendIntegrityError(tx_hash)
|
||||||
return v[1]
|
return v[1]
|
||||||
|
|
||||||
|
|
||||||
def upcoming(self, limit=0):
|
def upcoming(self, limit=0):
|
||||||
|
real_limit = 0
|
||||||
|
in_flight = 0
|
||||||
if limit > 0:
|
if limit > 0:
|
||||||
r = self.store.by_state(state=self.store.IN_NETWORK, not_state=self.store.FINAL)
|
in_flight = self.store.by_state(state=self.store.IN_NETWORK, not_state=self.store.FINAL)
|
||||||
limit -= len(r)
|
real_limit = limit - len(in_flight)
|
||||||
if limit <= 0:
|
if real_limit <= 0:
|
||||||
return []
|
return []
|
||||||
return self.store.upcoming(limit=limit)
|
r = self.store.upcoming(limit=real_limit)
|
||||||
|
logg.info('upcoming returning {} upcoming from limit {} less {} active in-flight txs'.format(len(r), limit, len(in_flight)))
|
||||||
|
return r
|
||||||
|
|
||||||
|
|
||||||
def pending(self):
|
def pending(self):
|
||||||
|
@ -5,28 +5,57 @@ import time
|
|||||||
# external imports
|
# external imports
|
||||||
from chainlib.status import Status as TxStatus
|
from chainlib.status import Status as TxStatus
|
||||||
from chainsyncer.filter import SyncFilter
|
from chainsyncer.filter import SyncFilter
|
||||||
from chainqueue.error import NotLocalTxError
|
from chainqueue.error import (
|
||||||
|
NotLocalTxError,
|
||||||
|
BackendIntegrityError,
|
||||||
|
)
|
||||||
|
from chaind.adapters.fs import ChaindFsAdapter
|
||||||
|
|
||||||
# local imports
|
# local imports
|
||||||
from .error import QueueLockError
|
from .error import QueueLockError
|
||||||
|
|
||||||
logg = logging.getLogger(__name__)
|
logg = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
class StateFilter(SyncFilter):
|
class StateFilter(SyncFilter):
|
||||||
|
|
||||||
delay_limit = 3.0
|
delay_limit = 3.0
|
||||||
|
|
||||||
def __init__(self, adapter, throttler=None):
|
def __init__(self, chain_spec, adapter_path, tx_adapter, throttler=None):
|
||||||
self.adapter = adapter
|
self.chain_spec = chain_spec
|
||||||
|
self.adapter_path = adapter_path
|
||||||
|
self.tx_adapter = tx_adapter
|
||||||
self.throttler = throttler
|
self.throttler = throttler
|
||||||
|
|
||||||
|
|
||||||
def filter(self, conn, block, tx, session=None):
|
def filter(self, conn, block, tx, session=None):
|
||||||
|
cache_tx = None
|
||||||
|
for i in range(3):
|
||||||
|
queue_adapter = None
|
||||||
try:
|
try:
|
||||||
cache_tx = self.adapter.get(tx.hash)
|
queue_adapter = ChaindFsAdapter(
|
||||||
|
self.chain_spec,
|
||||||
|
self.adapter_path,
|
||||||
|
self.tx_adapter,
|
||||||
|
None,
|
||||||
|
)
|
||||||
|
except BackendIntegrityError as e:
|
||||||
|
logg.error('adapter instantiation failed: {}, one more try'.format(e))
|
||||||
|
continue
|
||||||
|
|
||||||
|
try:
|
||||||
|
cache_tx = queue_adapter.get(tx.hash)
|
||||||
except NotLocalTxError:
|
except NotLocalTxError:
|
||||||
logg.debug('skipping not local transaction {}'.format(tx.hash))
|
logg.debug('skipping not local transaction {}'.format(tx.hash))
|
||||||
return False
|
return False
|
||||||
|
except BackendIntegrityError as e:
|
||||||
|
logg.error('adapter instantiation failed: {}, one more try'.format(e))
|
||||||
|
continue
|
||||||
|
|
||||||
|
break
|
||||||
|
|
||||||
|
if cache_tx == None:
|
||||||
|
raise NotLocalTxError(tx.hash)
|
||||||
|
|
||||||
delay = 0.01
|
delay = 0.01
|
||||||
while True:
|
while True:
|
||||||
@ -34,9 +63,9 @@ class StateFilter(SyncFilter):
|
|||||||
raise QueueLockError('The queue lock for tx {} seems to be stuck. Human meddling needed.'.format(tx.hash))
|
raise QueueLockError('The queue lock for tx {} seems to be stuck. Human meddling needed.'.format(tx.hash))
|
||||||
try:
|
try:
|
||||||
if tx.status == TxStatus.SUCCESS:
|
if tx.status == TxStatus.SUCCESS:
|
||||||
self.adapter.succeed(block, tx)
|
queue_adapter.succeed(block, tx)
|
||||||
else:
|
else:
|
||||||
self.adapter.fail(block, tx)
|
queue_adapter.fail(block, tx)
|
||||||
break
|
break
|
||||||
except QueueLockError as e:
|
except QueueLockError as e:
|
||||||
logg.debug('queue item {} is blocked, will retry: {}'.format(tx.hash, e))
|
logg.debug('queue item {} is blocked, will retry: {}'.format(tx.hash, e))
|
||||||
|
@ -104,7 +104,6 @@ class SessionController:
|
|||||||
logg.error('invalid input "{}"'.format(data_in_str))
|
logg.error('invalid input "{}"'.format(data_in_str))
|
||||||
raise ClientInputError()
|
raise ClientInputError()
|
||||||
|
|
||||||
logg.info('recv {} bytes'.format(len(data)))
|
|
||||||
return (srvs, data,)
|
return (srvs, data,)
|
||||||
|
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user