chore(dist): replace G1 config & handle other gc options by MingzhenHan · Pull Request #2664 · apache/hugegraph

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

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

edited

Loading

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

Reviewers

@imbajin imbajin imbajin approved these changes

@VGalaxies VGalaxies VGalaxies approved these changes

Assignees

No one assigned

Labels

lgtm

This PR has been approved by a maintainer

size:XS

This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Milestone

1.5.0

Development

Successfully merging this pull request may close these issues.

3 participants

@MingzhenHan @imbajin @VGalaxies