hashgraph / hedera-services

Crypto, token, consensus, file, and smart contract services for the Hedera public ledger
Apache License 2.0
265 stars 119 forks source link

chore: Extract throttle functionality from HandleContext #13974

Closed netopyr closed 1 week ago

netopyr commented 1 week ago

Description:

This PR extracts the throttle functionality from HandleContext.

Related issue(s):

Fixes #13972

Notes for reviewer:

Checklist

github-actions[bot] commented 1 week ago

Node: Unit Test Results

  1 551 files    1 551 suites   2h 57m 53s :stopwatch: 108 593 tests 108 534 :white_check_mark: 59 :zzz: 0 :x: 116 887 runs  116 828 :white_check_mark: 59 :zzz: 0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Node Death Reconnect) Results

3 tests   3 :white_check_mark:  6m 4s :stopwatch: 3 suites  0 :zzz: 3 files    0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Token) Results

 20 files   20 suites   5m 37s :stopwatch: 265 tests 265 :white_check_mark: 0 :zzz: 0 :x: 340 runs  340 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Restart) Results

3 tests   3 :white_check_mark:  6m 14s :stopwatch: 3 suites  0 :zzz: 3 files    0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Crypto) Results

 24 files   24 suites   14m 54s :stopwatch: 361 tests 361 :white_check_mark: 0 :zzz: 0 :x: 368 runs  368 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Time Consuming) Results

19 tests   19 :white_check_mark:  23m 42s :stopwatch:  4 suites   0 :zzz:  4 files     0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Misc) Results

 51 files   51 suites   21m 0s :stopwatch: 357 tests 357 :white_check_mark: 0 :zzz: 0 :x: 375 runs  375 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 1 week ago

Node: HAPI Test (Smart Contract) Results

 72 files   72 suites   22m 3s :stopwatch: 616 tests 616 :white_check_mark: 0 :zzz: 0 :x: 668 runs  668 :white_check_mark: 0 :zzz: 0 :x:

Results for commit 6a9588a2.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 82.05128% with 7 lines in your changes missing coverage. Please review.

Project coverage is 58.27%. Comparing base (9973fae) to head (6a9588a). Report is 63 commits behind head on develop.

Files Patch % Lines
...m/hedera/node/app/throttle/AppThrottleAdviser.java 85.29% 2 Missing and 3 partials :warning:
...p/workflows/handle/flow/DispatchHandleContext.java 66.66% 1 Missing :warning:
...ice/contract/impl/state/RootProxyWorldUpdater.java 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #13974 +/- ## ============================================= - Coverage 67.58% 58.27% -9.31% + Complexity 36486 22080 -14406 ============================================= Files 3732 2883 -849 Lines 149767 109662 -40105 Branches 15571 11159 -4412 ============================================= - Hits 101223 63910 -37313 + Misses 44179 42219 -1960 + Partials 4365 3533 -832 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.