[branch-2.1](multi-table-load) reduce routine load job transaction info storage in memory #31127
This commit is contained in:
committed by
GitHub
parent
9a708806e0
commit
d55d796c74
@ -1932,7 +1932,6 @@ public class FrontendServiceImpl implements FrontendService.Iface {
|
||||
}
|
||||
long timeoutMs = request.isSetThriftRpcTimeoutMs() ? request.getThriftRpcTimeoutMs() : 5000;
|
||||
List planFragmentParamsList = new ArrayList<>(tableNames.size());
|
||||
List<Long> tableIds = olapTables.stream().map(OlapTable::getId).collect(Collectors.toList());
|
||||
// todo: if is multi table, we need consider the lock time and the timeout
|
||||
boolean enablePipelineLoad = Config.enable_pipeline_load;
|
||||
try {
|
||||
@ -1950,8 +1949,6 @@ public class FrontendServiceImpl implements FrontendService.Iface {
|
||||
}
|
||||
multiTableFragmentInstanceIdIndexMap.put(request.getTxnId(), ++index);
|
||||
}
|
||||
Env.getCurrentGlobalTransactionMgr()
|
||||
.putTransactionTableNames(db.getId(), request.getTxnId(), tableIds);
|
||||
LOG.debug("receive stream load multi table put request result: {}", result);
|
||||
} catch (Throwable e) {
|
||||
LOG.warn("catch unknown result.", e);
|
||||
|
||||
@ -2246,17 +2246,6 @@ public class DatabaseTransactionMgr {
|
||||
return msgBuilder.toString();
|
||||
}
|
||||
|
||||
protected void putTransactionTableNames(long transactionId, List<Long> tableIds) {
|
||||
if (CollectionUtils.isEmpty(tableIds)) {
|
||||
return;
|
||||
}
|
||||
if (multiTableRunningTransactionTableIdMaps.contains(transactionId)) {
|
||||
multiTableRunningTransactionTableIdMaps.get(transactionId).addAll(tableIds);
|
||||
return;
|
||||
}
|
||||
multiTableRunningTransactionTableIdMaps.put(transactionId, tableIds);
|
||||
}
|
||||
|
||||
/**
|
||||
* Update transaction table ids by transaction id.
|
||||
* it's used for multi table transaction.
|
||||
|
||||
@ -776,9 +776,4 @@ public class GlobalTransactionMgr implements Writable {
|
||||
throws AnalysisException {
|
||||
getDatabaseTransactionMgr(dbId).updateMultiTableRunningTransactionTableIds(transactionId, tableIds);
|
||||
}
|
||||
|
||||
public void putTransactionTableNames(Long dbId, Long transactionId, List<Long> tableIds)
|
||||
throws AnalysisException {
|
||||
getDatabaseTransactionMgr(dbId).putTransactionTableNames(transactionId, tableIds);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user