danny0405 commented on PR #5445:
URL: https://github.com/apache/hudi/pull/5445#issuecomment-1111843176

   > > 
[api_patch.patch.zip](https://github.com/apache/hudi/files/8579506/api_patch.patch.zip)
 Thanks, i have write a patch, but i'm not sure whether we should go this 
direction, why not just use SQL directly.
   > > the reason
   > 
   > 1. some user only use datastream api to do complex operation such as 
operate flink state ,then they get a datastream which wanted to be insert into 
table
   > 2. read a hoodie table as a data stream will also be needed.because of do 
a complex operation or just they know well with datastream api
   > 
   > for your patch.LGTM only one question. RowData is only for table format 
instead of datastream. Row type will better in low-level APIs
   
   RowData is more efficient than Row, the flink connectors /formats all 
adapter to that. I think row data is enough.


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

Reply via email to