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: recordCache to commit added entries and implemented correctly the remove elements from the queue #10523
fix: recordCache to commit added entries and implemented correctly the remove elements from the queue #10523
Conversation
Node: E2E Test Results 1 files ± 0 1 suites ±0 22m 44s ⏱️ + 22m 44s Results for commit 1fb36db. ± Comparison against base commit b5574b1. This pull request removes 1 and adds 311 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## develop #10523 +/- ##
==========================================
Coverage 62.78% 62.79%
- Complexity 30938 30941 +3
==========================================
Files 3380 3382 +2
Lines 136732 136736 +4
Branches 14249 14249
==========================================
+ Hits 85852 85857 +5
+ Misses 47481 47476 -5
- Partials 3399 3403 +4 ☔ View full report in Codecov by Sentry. |
3476177
to
9a5700b
Compare
Node: HAPI Test (Crypto) Results211 tests ±0 209 ✔️ +1 21m 8s ⏱️ + 4m 35s Results for commit 1fb36db. ± Comparison against base commit b5574b1. This pull request skips 1 and un-skips 1 tests.
♻️ This comment has been updated with latest results. |
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.
One possible improvement, but not required.
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
4f8924c
to
fd6bbd0
Compare
fd6bbd0
to
82b36d6
Compare
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
...com/hedera/node/app/service/networkadmin/impl/test/handlers/NetworkAdminHandlerTestBase.java
Show resolved
Hide resolved
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
hedera-node/hedera-app/src/main/java/com/hedera/node/app/state/recordcache/RecordCacheImpl.java
Show resolved
Hide resolved
…s from history correctly Fix record cache to commit added entries to the queue state Ensure the record cache will also remove entries from the queue state when expired Created a "TruePredicate" class for when we require a predicate, but it should always be true, so we avoid duplicating the code yet again. This was needed for removing the head item from the queue state. Fix record cache to remove entries from histories when expired Fix tests as needed Adjust comments to match design and clarify the History map semantics. Signed-off-by: Lev Povolotsky <lev@swirldslabs.com> Signed-off-by: Joseph Sinclair <joseph.sinclair@swirldslabs.com>
82b36d6
to
5a131fe
Compare
…mpl-fails-to-remove-expired-transactions
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
3e1d9aa
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.com>
…ired-transactions
Signed-off-by: Michael Tinker <michael.tinker@swirldslabs.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.
LGTM!
[fix recordCache to commit added entries](fix: recordCache to commit added entries and implemented correctly the remove elements from the queue)
Related issue(s):
Fixes #10403
Fixes #10170