deardeng commented on code in PR #66680:
URL: https://github.com/apache/doris/pull/66680#discussion_r3766379842
##########
gensrc/thrift/DataSinks.thrift:
##########
@@ -319,6 +319,7 @@ struct TOlapTableSink {
// initial partition list is empty, so auto-partition tables whose first
partitions arrive at
// runtime still enter the correct mode from the start.
25: optional bool enable_adaptive_random_bucket
+ 26: optional map<string, i32> cross_az_succ_quorum
Review Comment:
The new Thrift fields are optional, so a new FE remains compatible with an
old coordinator BE without decoding, startup, or crash issues. During a rolling
upgrade, an old coordinator may not apply the AZ-aware wait and a load can be
rejected if a required AZ replica is slow. We consider this acceptable
transient behavior during the upgrade and do not plan to gate activation on
fleet-wide capability advertisement.
--
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]
