vinothchandar commented on pull request #2300:
URL: https://github.com/apache/hudi/pull/2300#issuecomment-749983667


   @garyli1019 if we write two log files, then we need a list somewhere , 
right?  Not sure how we would deal with that. I mean either WriteStatus has 
`List<WriteStat>` or WriteStat has  `List<Path>` for each file?
   
   On complexity of the change, if we simply change the following instances 
   
   ```
       List<HoodieWriteStat> writeStats = 
writeStatuses.map(WriteStatus::getStat).collect();
   
   ```
   
   to 
   
   ```
       List<HoodieWriteStat> writeStats = 
writeStatuses.flatMap(WriteStatus::getStats).collect();
   
   ```
   
   it's simple enough right? commit logic may not change much? I have to take a 
deeper pass, but let me know if I am overlooking something big. 


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


Reply via email to