deardeng commented on code in PR #66680:
URL: https://github.com/apache/doris/pull/66680#discussion_r3766375424
##########
fe/fe-core/src/main/java/org/apache/doris/planner/RemoteOlapTableSink.java:
##########
@@ -61,7 +60,7 @@ public TPaloNodesInfo createPaloNodesInfo() {
if (backend == null) {
continue;
}
- nodesInfo.addToNodes(new TNodeInfo(backend.getId(), 0,
backend.getHost(), backend.getBrpcPort()));
+ nodesInfo.addToNodes(createNodeInfo(backend));
Review Comment:
Remote Doris Insert is not covered by the BE-side cross-AZ quorum
optimization in this PR. The target FE still performs the authoritative
cross-AZ validation, so an invalid commit cannot succeed. In the worst case, a
remote insert may be rejected when a required target-AZ replica is slow. We can
address target metadata propagation separately if this availability
optimization is needed for remote inserts.
--
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]
