SteNicholas commented on code in PR #19681:
URL: https://github.com/apache/doris/pull/19681#discussion_r1198775706


##########
gensrc/thrift/PlanNodes.thrift:
##########
@@ -203,7 +204,7 @@ struct TBrokerScanRangeParams {
 
     // If partition_ids is set, data that doesn't in this partition will be 
filtered.
     8: optional list<i64> partition_ids
-    

Review Comment:
   No necessary change in this line. Revert.



##########
gensrc/thrift/PlanNodes.thrift:
##########
@@ -232,7 +233,7 @@ struct TEsScanRange {
   1: required list<Types.TNetworkAddress> es_hosts  //  es hosts is used by be 
scan node to connect to es
   // has to set index and type here, could not set it in scannode
   // because on scan node maybe scan an es alias then it contains one or more 
indices
-  2: required string index   

Review Comment:
   No necessary change in this line. Revert.



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