New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Retrieve all transaction receipts for a block in one request #6646
base: main
Are you sure you want to change the base?
Conversation
d89f198
to
ae09fcb
Compare
My node on snap sync shows static block reward as 0 even at genesis block of ETH mainnet. |
89ea57c
to
4341667
Compare
8d61d3e
to
ef71f57
Compare
6c1a93d
to
027d056
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the changes make sense. Would like a test for the new method on BlockchainQueries, think other changes are covered already.
* @param blockHash The hash of the block that corresponds to the receipts to retrieve. | ||
* @return The transaction receipts associated with the referenced block. | ||
*/ | ||
public Optional<List<TransactionReceiptWithMetadata>> transactionReceiptsByBlockHash( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to see a test for this method
is this comment related to this change or a separate issue? |
Separate issue. Incidentally discovered when testing the RPC method. |
ok - can you open an issue with more info if we need to look into it? thanks! |
@Wetitpig this PR needs updating with latest main - I don't have permission to update your branch |
Signed-off-by: Wetitpig <winsto003@hotmail.com>
Exclude burnt fees Signed-off-by: Wetitpig <winsto003@hotmail.com>
Signed-off-by: Wetitpig <winsto003@hotmail.com>
Signed-off-by: Wetitpig <winsto003@hotmail.com>
Signed-off-by: Wetitpig <winsto003@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Wetitpig can you add a test for BlockchainQueries. transactionReceiptsByBlockHash
Thanks for sending a pull request! Have you done the following?
doc-change-required
label to this PR if updates are required.Most advanced CI tests are deferred until PR approval, but you could:
./gradlew build
./gradlew acceptanceTest
./gradlew integrationTest
./gradlew ethereum:referenceTests:referenceTests
PR description
effectivePriorityFeePerGas
instead ofeffectiveGasPrice
becausebaseFee
would be burnt.