worker24h commented on a change in pull request #1173: load data from Parquet 
file (Issues #911)
URL: https://github.com/apache/incubator-doris/pull/1173#discussion_r285409289
 
 

 ##########
 File path: gensrc/thrift/PaloBrokerService.thrift
 ##########
 @@ -67,6 +67,7 @@ struct TBrokerListResponse {
 struct TBrokerOpenReaderResponse {
     1: required TBrokerOperationStatus opStatus;
     2: optional TBrokerFD fd;
+    3: optional i64 size; //file size
 
 Review comment:
   When read a parquet file by RPC, libparquet.a must kown the file size so 
that it can parse file. for example: When it read footer information.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to