fhan688 commented on PR #20486: URL: https://github.com/apache/hudi/pull/20486#issuecomment-6597405102
> Two non-blocking suggestions on comparison overhead and set ownership. Also, the PR description appears to describe the earlier revision: calculateOffsetDifference is now removed, and incomplete comparisons can still fail the known-offset lower-bound check. Please update the compatibility and behavior statements to match. Thanks! I’ve addressed both suggestions and updated the PR title and description to reflect the latest implementation, including the known-offset lower-bound behavior, removal of `calculateOffsetDifference()`, compatibility impact, and updated validation results. -- 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]
