sollhui opened a new pull request, #49516:
URL: https://github.com/apache/doris/pull/49516
### What problem does this PR solve?
Problem Summary:
**There are two questions when compile by AddressSanitizer:**
1. The RPC timeout did not meet expectations, as the three minutes timeout
actually exceeded in one minute.
2. The commit result is incorrect, transaction 21646315238602752. The commit
result is 21646508921529344. The origin stream load result:
```
{
"TxnId": 21646315238602752,
"Label": "188e8f85-1891-43bc-be39-6737da2e86d8",
"Comment": "",
"TwoPhaseCommit": "false",
"Status": "Fail",
"Message": "[ANALYSIS_ERROR]TStatus: errCode = 2, detailMessage = internal
error, commitTxn failed, transactionId:21646508921529344,
code:TXN_ALREADY_ABORTED, msg:transaction
[21646508921529344] is already aborted, db_id=1739017681561",
"NumberTotalRows": 5000,
"NumberLoadedRows": 5000,
"NumberFilteredRows": 0,
"NumberUnselectedRows": 0,
"LoadBytes": 589899,
"LoadTimeMs": 305186,
"BeginTxnTimeMs": 9,
"StreamLoadPutTimeMs": 9,
"ReadDataTimeMs": 12,
"WriteDataTimeMs": 2466,
"ReceiveDataTimeMs": 26,
"CommitAndPublishTimeMs": 0
}
```
**Reason:**
1. RPC timeout does not meet expectations: In ASAN mode, the Thrift
connection was reused, and the timeout will not be reset, so the RPC timeout
may be less than expectation timeout.
2. Chaotic commit result: In ASAN mode, the Thrift connection was reused.
Since this connection has not been reopened, if the following error occurred in
the last RPC:
```
errmsg=[THRIFT_RPC_ERROR]failed to call frontend service, FE
address=172.20.48.73:9020, reason: THRIFT_EAGAIN (timed out)
```
The result obtained by next load after multiplexing connection may be the
result of the previous one.
### 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: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
