nanjeshramesh commented on PR #19944: URL: https://github.com/apache/hudi/pull/19944#issuecomment-5809550814
@voonhous the nits from your earlier review are addressed (every `bf` is now `baseFile`, plus a checkstyle line-length fix that change caused), and Azure CI is green on the latest commit. Could you take another look when you have time? The AI review found no correctness issues. Its one suggestion is to use `baseFileToMerge` instead of `baseFile` in the merge handle constructors to keep the "being merged into" meaning of the old `oldDataFile` / `dataFileToBeMerged` names. I left that out to keep this a straight rename, but happy to do it if you would like it. -- 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]
