Add trace_id and status field values to ExtractGethTracesJob

blockchain-etl / ethereum-etl

Python scripts for ETL (extract, transform and load) jobs for Ethereum blocks, transactions, ERC20 / ERC721 tokens, transfers, receipts, logs, contracts, internal transactions. Data is available in Google BigQuery https://goo.gl/oY5BCQ

https://t.me/BlockchainETL

MIT License

2.94k stars 841 forks source link

Add trace_id and status field values to ExtractGethTracesJob #400

Open sajithaliyanage opened 1 year ago

sajithaliyanage commented 1 year ago

Closes https://github.com/blockchain-etl/ethereum-etl/issues/278

Also, This fix is needed to run the extract_contracts command since it checks the status to filter out contracts from the traces. Refer: https://github.com/blockchain-etl/ethereum-etl/blob/956695b77b37e9aafeff5a684d3bfa3a0dc3bdd7/ethereumetl/jobs/extract_contracts_job.py#L62

sajithaliyanage commented 1 year ago

@medvedev1088 please review and merge this PR. This is useful for extracting contracts from traces, as deriving contracts from receipts does not give all contracts in the blockchain. Similar fix is already merged in Ethereum ETL airflow dags https://github.com/blockchain-etl/ethereum-etl-airflow/issues/8

boshmaf commented 1 year ago

@medvedev1088 tried this fix and it works fine, could you please consider reviewing? Thanks @sajithaliyanage!