Skip to content

Comments

chore(dist): replace G1 config & handle other gc options#2664

Merged
VGalaxies merged 2 commits intoapache:masterfrom
MingzhenHan:removejdk8
Oct 10, 2024
Merged

chore(dist): replace G1 config & handle other gc options#2664
VGalaxies merged 2 commits intoapache:masterfrom
MingzhenHan:removejdk8

Conversation

@MingzhenHan
Copy link
Contributor

Subtask of #2575

If the user does not specify the GC method, g1 will be used by default.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 14, 2024
@imbajin imbajin changed the title chore(dist): set G1 as the default gc chore(dist): replace G1 config & handle other gc options Sep 14, 2024
@imbajin imbajin added this to the 1.5.0 milestone Sep 14, 2024
@codecov
Copy link

codecov bot commented Sep 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.71%. Comparing base (c88963c) to head (b4ce800).

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2664      +/-   ##
============================================
- Coverage     47.65%   43.71%   -3.95%     
+ Complexity      820      623     -197     
============================================
  Files           718      729      +11     
  Lines         58469    59536    +1067     
  Branches       7496     7663     +167     
============================================
- Hits          27866    26026    -1840     
- Misses        27815    30786    +2971     
+ Partials       2788     2724      -64     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 10, 2024
@VGalaxies VGalaxies merged commit 29ecc04 into apache:master Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants