docs: fix outdated content in deployment and actuator tutorial docs#1
Merged
Little-Peony merged 5 commits intodevelopfrom Apr 9, 2026
Merged
docs: fix outdated content in deployment and actuator tutorial docs#1Little-Peony merged 5 commits intodevelopfrom
Little-Peony merged 5 commits intodevelopfrom
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>
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
1 participant
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
implement-a-customized-actuator (EN+ZH)
SumContractfield number:52was already taken byMarketSellAssetContract; changed to60(next afterCancelAllUnfreezeV2Contract = 59)import "core/contract/math_contract.proto"step inapi.proto— without it the proto compiler cannot resolveSumContractin theInvokeSumRPC definitionstatic importforTRANSFER_FEEfromParameter.ChainConstant(not inherited fromAbstractActuator)@Before/@After→@BeforeClass/@AfterClass; Spring context was rebuilt per test methodArgsinitialization order:Args.setParam()must be called beforeArgs.getInstance()to avoid passing a stale instance toinitServices()"output-directory"path with@ClassRule TemporaryFolderaddService()/initServices()/startServices()/shutdownServices()calls with the actualApplication.startup()/shutdown()APITransactionRegisterauto-discoversAbstractActuatorsubclasses inorg.tron.core.actuatorat startup via Reflections — no manual registration neededprotoc v3.4.0note; point readers tobuild.gradlefor the version@Rule Timeoutto guard against hanging testsmodular-deployment (EN+ZH)
java -jar FullNode.jar; README shows it as a fully supported launch method*.bat/ Windows startup instructions; README line 39 explicitly states Windows is not supported-XX:+UseConcMarkSweepGC,-XX:+CMSParallelRemarkEnabled,-XX:+CMSScavengeBeforeRemark) with JDK 8/17-compatible options; using the old flags causes JVM startup failure on JDK 17 (required for ARM64)modular-introduction (EN+ZH)
cryptoandpluginsmodule descriptionsTest plan
SumContract = 60does not conflict with any existingContractTypevalueapi.protoimportvmoptionsexample on JDK 8 and JDK 17🤖 Generated with Claude Code