loukey-lj commented on code in PR #7336: URL: https://github.com/apache/hudi/pull/7336#discussion_r1049198455
########## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/io/HoodieWriteHandle.java: ########## @@ -86,6 +88,7 @@ protected HoodieTimer timer; protected WriteStatus writeStatus; + protected HoodieRecordLocation newRecordLocation; Review Comment: I got it wrong. I thought that a WriteStatus only has one location instance. Now it seems that there is no great change from obtaining the loaction from HoodieRecord. But in some partition change scenarios, we may need to use HoodieRecord's operation to determine the curd of the index -- 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org