sollhui commented on code in PR #64356:
URL: https://github.com/apache/doris/pull/64356#discussion_r3392890037


##########
gensrc/thrift/FrontendService.thrift:
##########
@@ -1425,6 +1427,8 @@ struct TReplacePartitionRequest {
     5: optional string be_endpoint
     6: optional bool write_single_replica = false
     7: optional Types.TUniqueId query_id
+    // Whether the caller's table sink is using load_to_single_tablet mode.
+    8: optional bool load_to_single_tablet = false

Review Comment:
   This is a request-side flag, not a result field.
   
   The result already carries the actual routing value through
   TOlapTablePartition.load_tablet_idx. FE needs this request flag to know 
whether
   the caller sink is running in load_to_single_tablet/FIND_TABLET_EVERY_SINK 
mode
   when it creates or replaces runtime auto partitions.
   
   Without this flag, createPartition()/replacePartition() only know the table 
and
   partition metadata, but not the original sink routing mode. Then FE cannot 
decide
   whether it should assign/cache load_tablet_idx for the newly created/replaced
   random partition.



-- 
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]

Reply via email to