fix(doc): update x86 java start params #6478
Merged
+24
−29
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sunny6889
commented
Nov 10, 2025
bladehan1
reviewed
Nov 10, 2025
added 2 commits
November 10, 2025 10:51
Sunny6889
commented
Nov 10, 2025
…/java-tron into release_v4.8.1_fix_jvm # Conflicts: # README.md
kuny0707
approved these changes
Nov 18, 2025
6 tasks
| ### ARM64 (JDK 17) | ||
| ```bash | ||
| $ nohup java -Xms9G -Xmx9G -XX:+UseZGC \ | ||
| $ nohup java -Xmx9G -XX:+UseZGC \ |
Contributor
There was a problem hiding this comment.
- Why delete
-Xms9Gfor ZGC just for fullnode? - If so, the same changes are required for SR and
start.sh.sample
Author
There was a problem hiding this comment.
Because in normal case FullNode don't need minimum 9G，as JDK 17 use RocksDB, which will take more space. It is not directly related with ZGC.
Author
There was a problem hiding this comment.
SR don't need change, as the minimum requirement for SR machine is 64G, so it does not matter. This remove Xms9G is used more for 16G machines.
Author
There was a problem hiding this comment.
But I changed start.sh.sample
lxcmyf
approved these changes
Nov 20, 2025
Sunny6889
force-pushed
the
release_v4.8.1_fix_jvm
branch
2 times, most recently
from
2be3670 to
86ac0a7
Compare
Squashed commits: [fe4369f510] revert (+1 squashed commit) Squashed commits: [1925a2d1f1] revert (+1 squashed commit) Squashed commits: [336092cd1a] feat(dependencies): bump libp2p from 2.2.6 to 2.2.7 (tronprotocol#6481) * update libp2p from 2.2.6 to 2.2.7 * add pom file (+4 squashed commits) Squashed commits: [d0bdaf0] revert changes [6b14bdb] JDK 8 [d279184] modify start.sh.simple [e24693a] update (+1 squashed commit) Squashed commits: [b54aed3] revert commitee config for proposalExpireTime (+1 squashed commit) Squashed commits: [e132811] update config for proposalExpireTime
Sunny6889
force-pushed
the
release_v4.8.1_fix_jvm
branch
from
86ac0a7 to
c90a254
Compare
halibobo1205
approved these changes
Nov 20, 2025
lvs0075
approved these changes
Nov 25, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
Why are these changes required?
-Xmx 12Gis safe for a simple FullNode even when blocks fail to solidify promptly—even with a maximum wait of one maintenance window.-Xmxin JDK 17 because in normal case FullNode don't need minimum 9G，as JDK 17 use RocksDB, which will take more space.This PR has been tested by:
Follow up
Extra details