deardeng opened a new pull request, #58714:
URL: https://github.com/apache/doris/pull/58714
Currently, tablet report logic uses a ForkJoinPool to process tablet
information, but often encounters unexplained hangs in the ForkJoinPool. The
printed stack trace doesn't reveal where the hang occurs, making it difficult
to troubleshoot the issue.
use forkjoin pool, report stuck stack such as
```
"report-thread" #187 daemon prio=5 os_prio=0 cpu=97864.95ms
elapsed=2469428.50s tid=0x00007ff1cb5c8530 nid=0xef2 waiting on condition
[0x00007fef462e1000]
java.lang.Thread.State: WAITING (parking)
at jdk.internal.misc.Unsafe.park([email protected]/Native Method)
- parking to wait for <0x00000006dc3fae00> (a
java.util.concurrent.ForkJoinTask$AdaptedRunnableAction)
at
java.util.concurrent.locks.LockSupport.park([email protected]/LockSupport.java:341)
at
java.util.concurrent.ForkJoinTask.awaitDone([email protected]/ForkJoinTask.java:468)
at
java.util.concurrent.ForkJoinTask.join([email protected]/ForkJoinTask.java:670)
at
org.apache.doris.catalog.TabletInvertedIndex.tabletReport(TabletInvertedIndex.java:370)
at
org.apache.doris.master.ReportHandler.tabletReport(ReportHandler.java:509)
at
org.apache.doris.master.ReportHandler$ReportTask.exec(ReportHandler.java:339)
at
org.apache.doris.master.ReportHandler.runOneCycle(ReportHandler.java:1466)
at org.apache.doris.common.util.Daemon.run(Daemon.java:119)
Locked ownable synchronizers:
- None
```
Can't find where the problem is in this stack
When the tablet report is stuck, the TabletInvertedIndex holds a read lock,
leading to a deadlock.
This pr uses a normal thread pool to replace forkjoinpool
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
