-
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
Fix importer workflow timing out #2501
Conversation
Codecov Report
@@ Coverage Diff @@
## main #2501 +/- ##
============================================
+ Coverage 91.51% 91.55% +0.03%
+ Complexity 2527 2526 -1
============================================
Files 421 421
Lines 11709 11741 +32
Branches 1018 1018
============================================
+ Hits 10716 10749 +33
+ Misses 649 648 -1
Partials 344 344
Continue to review full report at Codecov.
|
6b0599d
to
c77661b
Compare
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
fb9a392
to
05a8cc0
Compare
Kudos, SonarCloud Quality Gate passed! |
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
Description:
.mvn/jvm.config
to configure default JVM build settings.mvn/maven.config
to configure default Maven build parameters--threads 1C
to support building modules in parallellightweight
tohttpclient
for better retry configurabilityRelated issue(s):
Notes for reviewer:
Using a parallel build improves
./mvnw verify
runtime from 11 minutes to 8 minutes on my machine.Checklist