hashgraph / hedera-services

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

feat: Cherry-Pick (0.55): Define throttle for GetBalance queries #16339

Closed vtronkov closed 4 days ago

vtronkov commented 4 days ago

PR to develop PR to 0.56

codacy-production[bot] commented 4 days ago

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
:white_check_mark: +0.00% (target: -1.00%) :white_check_mark:
Coverage variation details | | Coverable lines | Covered lines | Coverage | | ------------- | ------------- | ------------- | ------------- | | Common ancestor commit (86625a137afbd4c16a1f70054503d68aadbfddcb) | 109350 | 67527 | 61.75% | | | Head commit (00d49a26a0512a6cfe782a755fbc4c05ed025ac3) | 109350 (+0) | 67527 (+0) | 61.75% (**+0.00%**) | **Coverage variation** is the difference between the coverage for the head and common ancestor commits of the pull request branch: ` - `
Diff coverage details | | Coverable lines | Covered lines | Diff coverage | | ------------- | ------------- | ------------- | ------------- | | Pull request (#16339) | 0 | 0 | **∅ (not applicable)** | **Diff coverage** is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: `/ * 100%`

See your quality gate settings    Change summary preferences

Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more

codecov[bot] commented 4 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.21%. Comparing base (86625a1) to head (00d49a2). Report is 1 commits behind head on release/0.55.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339/graphs/tree.svg?width=650&height=150&src=pr&token=ZPMV8C93DV&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph)](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) ```diff @@ Coverage Diff @@ ## release/0.55 #16339 +/- ## =============================================== Coverage 58.21% 58.21% Complexity 21617 21617 =============================================== Files 2784 2784 Lines 109533 109533 Branches 11211 11211 =============================================== Hits 63761 63761 Misses 41896 41896 Partials 3876 3876 ``` | [Files with missing lines](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph) | Coverage Δ | | |---|---|---| | [.../java/com/hedera/node/config/data/StatsConfig.java](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339?src=pr&el=tree&filepath=hedera-node%2Fhedera-config%2Fsrc%2Fmain%2Fjava%2Fcom%2Fhedera%2Fnode%2Fconfig%2Fdata%2FStatsConfig.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph#diff-aGVkZXJhLW5vZGUvaGVkZXJhLWNvbmZpZy9zcmMvbWFpbi9qYXZhL2NvbS9oZWRlcmEvbm9kZS9jb25maWcvZGF0YS9TdGF0c0NvbmZpZy5qYXZh) | `100.00% <ø> (ø)` | | [![Impacted file tree graph](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339/graphs/tree.svg?width=650&height=150&src=pr&token=ZPMV8C93DV&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph)](https://app.codecov.io/gh/hashgraph/hedera-services/pull/16339?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=hashgraph)
github-actions[bot] commented 4 days ago

Node: HAPI Test (Restart) Results

8 files  1 errors  7 suites   7m 47s :stopwatch: 6 tests 6 :white_check_mark: 0 :zzz: 0 :x: 7 runs  7 :white_check_mark: 0 :zzz: 0 :x:

For more details on these parsing errors, see this check.

Results for commit 00d49a26.

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

mhess-swl commented 4 days ago

Hedera Node JRS Panel tests don't appear related to the additional throttle added in this PR. Merging