[fix](insert-into) fix insert into lose data (#29802)
This commit is contained in:
committed by
yiguolei
parent
01a91a482e
commit
223e466514
@ -2520,7 +2520,14 @@ public class Coordinator implements CoordInterface {
|
||||
params.getBackendId(), status.getErrorMsg());
|
||||
updateStatus(status, params.getFragmentInstanceId());
|
||||
}
|
||||
if (ctx.fragmentInstancesMap.get(params.fragment_instance_id).getIsDone()) {
|
||||
|
||||
// params.isDone() should be promised.
|
||||
// There are some periodic reports during the load process,
|
||||
// and the reports from the intermediate process may be concurrent with the last report.
|
||||
// The last report causes the counter to decrease to zero,
|
||||
// but it is possible that the report without commit-info triggered the commit operation,
|
||||
// resulting in the data not being published.
|
||||
if (ctx.fragmentInstancesMap.get(params.fragment_instance_id).getIsDone() && params.isDone()) {
|
||||
if (params.isSetDeltaUrls()) {
|
||||
updateDeltas(params.getDeltaUrls());
|
||||
}
|
||||
@ -2585,7 +2592,13 @@ public class Coordinator implements CoordInterface {
|
||||
}
|
||||
}
|
||||
|
||||
if (execState.done) {
|
||||
// params.isDone() should be promised.
|
||||
// There are some periodic reports during the load process,
|
||||
// and the reports from the intermediate process may be concurrent with the last report.
|
||||
// The last report causes the counter to decrease to zero,
|
||||
// but it is possible that the report without commit-info triggered the commit operation,
|
||||
// resulting in the data not being published.
|
||||
if (execState.done && params.isDone()) {
|
||||
if (params.isSetDeltaUrls()) {
|
||||
updateDeltas(params.getDeltaUrls());
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user