-
Notifications
You must be signed in to change notification settings - Fork 117
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
fix: Cherry-pick: Avoid catastrophic failure for too large transactions (#13029) #13079
Conversation
Signed-off-by: Michael Heinrichs <netopyr@users.noreply.github.com> (cherry picked from commit 8ef29c4)
Node: HAPI Test (Restart) Results2 tests 2 ✅ 6m 21s ⏱️ Results for commit 6d52de0. |
Node: HAPI Test (Node Death Reconnect) Results1 tests 1 ✅ 17s ⏱️ For more details on these parsing errors, see this check. Results for commit 6d52de0. |
Node: HAPI Test (Token) Results228 tests 227 ✅ 19m 43s ⏱️ Results for commit 6d52de0. |
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.
LGTM! Thanks @netopyr
Node: HAPI Test (Crypto) Results335 tests 335 ✅ 39m 16s ⏱️ Results for commit 6d52de0. |
Node: HAPI Test (Misc) Results457 tests 447 ✅ 39m 50s ⏱️ For more details on these parsing errors, see this check. Results for commit 6d52de0. ♻️ This comment has been updated with latest results. |
Node: HAPI Test (Time Consuming) Results21 tests 21 ✅ 54m 9s ⏱️ Results for commit 6d52de0. |
Node: Unit Test Results 2 267 files ±0 2 267 suites ±0 2h 56m 29s ⏱️ - 57m 25s Results for commit 6d52de0. ± Comparison against base commit 161bb5d. This pull request removes 3993 and adds 3757 tests. Note that renamed tests count towards both.
|
Node: HAPI Test (Smart Contract) Results587 tests 587 ✅ 1h 5m 59s ⏱️ Results for commit 6d52de0. |
Description:
Cherry-pick of #13029
Related issue(s):
Closes #13004