Fix and restructure java-tron Core Modules doc#349
Open
abn2357 wants to merge 3 commits into
Open
Conversation
- Attribute TRON-layer handshake (P2P_HELLO) and keep-alive (P2P_PING/PONG) to core/net instead of libp2p; libp2p only covers low-level peer discovery and connection transport - Correct protocol/chainbase directory paths in code blocks - Update network summary to reflect discovery/connection moved to libp2p - Fix architecture diagram label: crypt -> crypto
…ives The single code-structure page bundled three independent articles (each with its own summary). Split the two deep-dive sections into standalone pages: - developers/chainbase.md: ChainBase deep dive - developers/network.md: P2P Network deep dive The overview page keeps the 7-module map and links out to both deep dives. Add a ChainBase/P2P Network entry to the developer nav (right after Core Modules), and cross-link the P2P Network deep dive with the operational 'Connect to the TRON Network' guide.
vivian1912
reviewed
Jun 26, 2026
|
|
||
| 节点接收到新区块后产生的变更不会直接存储到持久化存储中，而是首先存在 snapshotImpl 中，每接收一个区块对应产生一个snapshotImpl 不断的接收区块会导致 snapshotImpl 越来越多，什么时候会写入到持久化存储中呢？ | ||
|
|
||
| SnapshotManager 中存在两个变量：size 和 maxSize，size 此处我们简单理解为目前内存中有多少层 snapshotImpl，maxSIze 则表示目前固化块和最新块高度的差值。 |
Contributor
There was a problem hiding this comment.
Original typo remains: maxSIze -> maxSize
vivian1912
approved these changes
Jun 26, 2026
Benson0224
approved these changes
Jun 27, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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
Two related changes to
developers/code-structure.mdand the developer docs:1. Accuracy fixes (verified against java-tron source)
P2P_HELLO) and keep-alive (P2P_PING/P2P_PONG) tocore/netinstead of libp2p; libp2p only covers low-level peer discovery and connection transport.db/provides theRevokingDatabaseinterface andTronStoreWithRevokingbase class; the actual rollback implementation isdb2/core/SnapshotManager(the oldAbstractRevokingStoreno longer exists).protocol/src/main/protos,common/src/main/java/org/tron,chainbase/src/main/java/org/tron) and alignapi//core/descriptions with the code block.crypt->crypto.2. Restructure
The single page bundled three independent articles (each with its own summary). Split the two deep-dive sections into standalone pages:
developers/chainbase.md— ChainBase deep divedevelopers/network.md— P2P Network deep diveThe overview page keeps the 7-module map and links out to both deep dives. The developer nav gains a ChainBase / P2P Network entry right after Core Modules, and the P2P Network deep dive is cross-linked with the operational "Connect to the TRON Network" guide.
Test plan
mkdocs buildsucceeds with no warnings or errors#block-synchronization,#block-and-transaction-broadcast) resolve correctly