feat(event): optimize event service thread close logic by zeusoo001 · Pull Request #6463 · tronprotocol/java-tron
Conversation
What does this PR do?
Optimize event service thread close logic.
Why are these changes required?
This PR has been tested by:
- Unit Tests
- Manual Testing
Follow up
Extra details
| long endNum = manager.getDynamicPropertiesStore().getLatestSolidifiedBlockNum(); | ||
| while (tmp < endNum) { | ||
| if (thread.isInterrupted()) { | ||
| if (thread.isInterrupted() || isClosed) { |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is a good resource leakage fix. I wonder how you find it?
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It may throw exception when the program closes:
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good
halibobo1205 pushed a commit to halibobo1205/java-tron that referenced this pull request
Oct 22, 2025
halibobo1205 pushed a commit to halibobo1205/java-tron that referenced this pull request
Oct 22, 2025
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