-
-
Notifications
You must be signed in to change notification settings - Fork 862
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
Use 1k buffer in FileUtils.readLines to reduce heap allocation pressure #2612
Conversation
JMH results with various buffer sizes:
The gains from going below 1k appear to be very small, but the delta from the default 8k to 1k is big: from |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2612 +/- ##
==========================================
+ Coverage 86.23% 86.84% +0.60%
==========================================
Files 30 30
Lines 1337 1338 +1
Branches 189 189
==========================================
+ Hits 1153 1162 +9
- Misses 103 104 +1
+ Partials 81 72 -9 ☔ View full report in Codecov by Sentry. |
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 with one question
No description provided.