Uh oh!
There was an error while loading. Please reload this page.
[fix](txn) Fix coordidator be restart not abort txn - #35342
Conversation
doris-robot
commented
May 24, 2024
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
clang-tidy review says "All clean, LGTM! 👍" |
yujun777
commented
May 24, 2024
run buildall |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
doris-robot
commented
May 24, 2024
TPC-H: Total hot run time: 40765 ms |
doris-robot
commented
May 24, 2024
TPC-DS: Total hot run time: 172652 ms |
doris-robot
commented
May 24, 2024
ClickBench: Total hot run time: 30.08 s |
doris-robot
commented
May 24, 2024
TeamCity be ut coverage result: |
yujun777
commented
May 27, 2024
run p0 |
dataroaring
left a comment
There was a problem hiding this comment.
- httpstream should be handled
- Test cases should include insert into values and what about if a fe is restated.
| } | ||
| request.__set_request_id(ctx->id.to_thrift()); | ||
| request.__set_backend_id(_exec_env->master_info()->backend_id); | ||
There was a problem hiding this comment.
httpstream shoud also be considered.
dataroaring
commented
May 27, 2024
|
0dc1c1d to
7adb998Compareclang-tidy review says "All clean, LGTM! 👍" |
yujun777
commented
Jun 6, 2024
run buildall |
doris-robot
commented
Jun 6, 2024
TPC-H: Total hot run time: 41105 ms |
doris-robot
commented
Jun 6, 2024
TPC-DS: Total hot run time: 172354 ms |
doris-robot
commented
Jun 6, 2024
ClickBench: Total hot run time: 30.4 s |
doris-robot
commented
Jun 6, 2024
TeamCity be ut coverage result: |
clang-tidy review says "All clean, LGTM! 👍" |
yujun777
commented
Jun 11, 2024
run buildall |
doris-robot
commented
Jun 11, 2024
TPC-H: Total hot run time: 40701 ms |
doris-robot
commented
Jun 11, 2024
TPC-DS: Total hot run time: 173546 ms |
doris-robot
commented
Jun 11, 2024
ClickBench: Total hot run time: 30.97 s |
doris-robot
commented
Jun 11, 2024
TeamCity be ut coverage result: |
| @SerializedName(value = "ip") | ||
| public String ip; | ||
| // frontend/backend start time | ||
| @SerializedName(value = "startTime") |
There was a problem hiding this comment.
short name is preferred, like st.
03abb17 to
daf6f98Compareclang-tidy review says "All clean, LGTM! 👍" |
1 similar comment
clang-tidy review says "All clean, LGTM! 👍" |
yujun777
commented
Jun 14, 2024
run buildall |
doris-robot
commented
Jun 14, 2024
TPC-H: Total hot run time: 40520 ms |
doris-robot
commented
Jun 14, 2024
TPC-DS: Total hot run time: 174151 ms |
doris-robot
commented
Jun 14, 2024
ClickBench: Total hot run time: 30.55 s |
doris-robot
commented
Jun 14, 2024
TeamCity be ut coverage result: |
PR approved by at least one committer and no changes requested. |
BUG: fe will abort coordidator BE's txn when be shutdown exceeds 5min. But if BE restart within 5min, then this BE's txns will not abort until timeout. FIX: every txn will record BE's id and its start time. When fe found be's startTime change, it will abort the BE's old txns.
…pache#28662, apache#35342) (apache#39317) cherry-pick: apache#22781, apache#28662, apache#35342 --------- Co-authored-by: HHoflittlefish777 <77738092+HHoflittlefish777@users.noreply.github.com>
BUG: fe will abort coordidator BE's txn when be shutdown exceeds 5min. But if BE restart within 5min, then this BE's txns will not abort until timeout. FIX: every txn will record BE's id and its start time. When fe found be's startTime change, it will abort the BE's old txns.
BUG: fe will abort coordidator BE's txn when be shutdown exceeds 5min. But if BE restart within 5min, then this BE's txns will not abort until timeout.
FIX: every txn will record BE's id and its start time. When fe found be's startTime change, it will abort the BE's old txns.
Proposed changes
Issue Number: close #xxx
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...