docs: fix outdated content in deployment and actuator tutorial docs#6662
Open
Little-Peony wants to merge 5 commits intotronprotocol:developfrom
Open
docs: fix outdated content in deployment and actuator tutorial docs#6662Little-Peony wants to merge 5 commits intotronprotocol:developfrom
Little-Peony wants to merge 5 commits intotronprotocol:developfrom
Conversation
implement-a-customized-actuator (EN+ZH): - Fix test lifecycle: @Before/@after → @BeforeClass/@afterclass so the Spring context is created only once per test class, not per test method - Fix Args initialization order: Args.setParam() must be called before Args.getInstance() to avoid passing a stale unconfigured instance to appTest.initServices() - Replace hardcoded dbPath with @ClassRule TemporaryFolder to keep test directories isolated and auto-cleaned - Add @rule Timeout to guard against hangs - Remove the specific protoc v3.4.0 version note; point readers to the version declared in build.gradle instead modular-deployment (EN+ZH): - Replace CMS GC flags (-XX:+UseConcMarkSweepGC, -XX:+CMSParallelRemarkEnabled, -XX:+CMSScavengeBeforeRemark) that were removed in JDK 14 and would cause startup failures on JDK 17 (required for ARM64) - Add -Xms/-Xmx to the example and note JDK 8/17 compatibility modular-introduction (EN+ZH): - Update module count from six to eight - Add descriptions for the crypto and plugins modules Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
implement-a-customized-actuator (EN+ZH): - SumContract field number: 52 is already taken by MarketSellAssetContract; change to 60 (next available number after CancelAllUnfreezeV2Contract=59) - SumActuator: add missing static import for TRANSFER_FEE from Parameter.ChainConstant (not inherited from AbstractActuator) - SumActuatorTest: Application does not have addService()/initServices()/ startServices()/shutdownServices(); replace with startup()/shutdown() which are the actual API methods Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
implement-a-customized-actuator (EN+ZH): - Add the missing 'import "core/contract/math_contract.proto"' step in api.proto; without it the proto compiler cannot resolve SumContract in the InvokeSum RPC definition (compile failure) - Add a note explaining that SumActuator must live in org.tron.core.actuator: TransactionRegister.registerActuator() uses Reflections to scan that package at startup and auto-registers every AbstractActuator subclass via the super() constructor call into TransactionFactory Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…t guide modular-deployment (EN+ZH): - Remove 'will be deprecated' note for java -jar FullNode.jar: README shows it as a fully supported launch method alongside the script - Add explicit platform note: Linux and macOS only, Windows not supported - Remove '*.bat on Windows' startup instruction: Windows is not supported per README line 39 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
warku123
approved these changes
Apr 9, 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
2 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.
Summary
Documentation cleanup across all six doc files under
docs/. Changes are purely documentation — no production code modified.implement-a-customized-actuator (EN + ZH)
SumContractfield number: changed52→60; field 52 is already taken byMarketSellAssetContractimport "core/contract/math_contract.proto"toapi.proto— without it the proto compiler cannot resolveSumContractin theInvokeSumRPC definitionimport static org.tron.core.config.Parameter.ChainConstant.TRANSFER_FEE; this constant is not inherited fromAbstractActuator@Before/@After→@BeforeClass/@AfterClass; the Spring context was rebuilt for every single test methodArgsinitialization order:Args.setParam()must be called beforeArgs.getInstance(); the original code passed a stale unconfigured instance toinitServices()"output-directory": replaced with@ClassRule TemporaryFolderaddService(),initServices(),startServices(),shutdownServices()do not exist onApplication; replaced withstartup()/shutdown()TransactionRegister.registerActuator()uses Reflections to scanorg.tron.core.actuatorat startup and auto-registers allAbstractActuatorsubclasses — no manual registration needed as long as the class is in that packageprotoc v3.4.0note; readers should use the version inbuild.gradle@Rule Timeout: guard against hanging testsmodular-deployment (EN + ZH)
java -jar FullNode.jaris still a fully supported launch method (shown in README); removed the "will be deprecated" claim*.batstartup instructions and added a platform note (Linux / macOS only)-XX:+UseConcMarkSweepGC,-XX:+CMSParallelRemarkEnabled,-XX:+CMSScavengeBeforeRemarkwere removed in JDK 14; using them causes JVM startup failure on JDK 17 (required for ARM64). Replaced with JDK 8/17-compatible optionsmodular-introduction (EN + ZH)
cryptoandpluginsmodule descriptionsTest plan
SumContract = 60does not conflict with any existingContractTypevalue inTron.protoapi.protowith the new importvmoptionsexample on JDK 8 and JDK 17🤖 Generated with Claude Code