Skip to content

Commit

Permalink
Merge pull request #395 from blockchain-etl/feat/remove-quickfix-traces
Browse files Browse the repository at this point in the history
Remove TempFix for insufficient funds since this has been resolved on the node
  • Loading branch information
medvedev1088 committed Sep 23, 2022
2 parents e1f658b + 3ed8b8b commit 533f516
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 12 deletions.
2 changes: 1 addition & 1 deletion ethereumetl/cli/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@


@click.group()
@click.version_option(version='2.0.5')
@click.version_option(version='2.0.6')
@click.pass_context
def cli(ctx):
pass
Expand Down
11 changes: 1 addition & 10 deletions ethereumetl/jobs/export_traces_job.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,16 +87,7 @@ def _export_batch(self, block_number_batch):

# TODO: Change to traceFilter when this issue is fixed
# https://github.com/paritytech/parity-ethereum/issues/9822
try:
json_traces = self.web3.parity.traceBlock(block_number)
except ValueError as e:
if 'insufficient funds' in str(e):
logging.exception(f'An "insufficient funds" error occurred while tracing block {block_number}. '
f'See here for more details: https://github.com/ledgerwatch/erigon/issues/5284. '
f'The block will be skipped.')
return
else:
raise e
json_traces = self.web3.parity.traceBlock(block_number)

if json_traces is None:
raise ValueError('Response from the node is None. Is the node fully synced? Is the node started with tracing enabled? Is trace_block API enabled?')
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ def read(fname):

setup(
name='ethereum-etl',
version='2.0.5',
version='2.0.6',
author='Evgeny Medvedev',
author_email='evge.medvedev@gmail.com',
description='Tools for exporting Ethereum blockchain data to CSV or JSON',
Expand Down

0 comments on commit 533f516

Please sign in to comment.