fix(testcase): rpc close leakage by Sunny6889 · Pull Request #6483 · tronprotocol/java-tron

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

@Sunny6889

Copy link

Contributor

@Sunny6889 Sunny6889 commented

Nov 21, 2025

edited

Loading

What does this PR do?

  • Fix apiServer shutdown close executorService leakage
  • Make RpcApiServicesTest channel usage as recommended, more reliable

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@Sunny6889 Sunny6889 changed the title fix rpc close leakage fix(testcase): rpc close leakage

Nov 27, 2025

@kuny0707 kuny0707 moved this to In Review in java-tron

Nov 27, 2025

@kuny0707 kuny0707 merged commit 8aed63c into tronprotocol:release_v4.8.1

Nov 27, 2025

7 checks passed

@github-project-automation github-project-automation bot moved this from In Review to Done in java-tron

Nov 27, 2025

warku123 pushed a commit to warku123/java-tron that referenced this pull request

Feb 6, 2026

@Sunny6889 Sunny6889 deleted the release_v4.8.1_fix_rpc branch

March 23, 2026 07:59

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kuny0707 kuny0707 kuny0707 approved these changes

@lvs0075 lvs0075 lvs0075 approved these changes

+4 more reviewers

@Federico2014 Federico2014 Federico2014 approved these changes

@halibobo1205 halibobo1205 halibobo1205 approved these changes

@bladehan1 bladehan1 bladehan1 approved these changes

@barbatos2011 barbatos2011 barbatos2011 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@Sunny6889 @kuny0707 @Federico2014 @halibobo1205 @bladehan1 @barbatos2011 @lvs0075