-
Notifications
You must be signed in to change notification settings - Fork 108
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
Improve monitor startup performance (0.38.1) #2397
Conversation
* Improve monitor startup performance by moving publish and subscribe flow initialization off main thread * Increase node validation timeout to 30s * Adjust monitor probe delay to reflect startup improvement Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Codecov Report
@@ Coverage Diff @@
## release/0.38 #2397 +/- ##
==================================================
- Coverage 84.35% 84.32% -0.04%
+ Complexity 2309 2308 -1
==================================================
Files 439 439
Lines 11982 11983 +1
Branches 1020 1020
==================================================
- Hits 10108 10105 -3
- Misses 1556 1560 +4
Partials 318 318
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
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
@@ -186,8 +186,8 @@ private boolean validateNode(Client client, NodeProperties node) { | |||
new TransferTransaction() | |||
.addHbarTransfer(nodeAccountId, hbar) | |||
.addHbarTransfer(client.getOperatorAccountId(), hbar.negated()) | |||
.execute(client, Duration.ofSeconds(10L)) | |||
.getReceipt(client, Duration.ofSeconds(10L)); | |||
.execute(client, Duration.ofSeconds(30L)) |
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.
nit: if you circle - move to shared constant.
Meant to say this in main before you merged.
Description:
Related issue(s):
Notes for reviewer:
Cherry pick of #2395 to 0.38.1
Checklist