zhang-arvin opened a new pull request, #930: URL: https://github.com/apache/incubator-xtable/pull/930
Fixes #928 ## What changes were proposed in this pull request? `DeltaKernelConversionSource.getCommitsBacklog` computed the incremental range as `start = versionAtLastSyncInstant + 1`, `end = latestVersion` with no `start <= end` guard. With no new commits since the last sync, the range became `[n+1, n]` and Delta Kernel's `getChanges` rejected the inverted interval with `Invalid version range`, breaking the documented pattern of calling sync after every write / on a schedule. The fix detects `versionAtLastSyncInstant >= latestVersion`, logs that there is nothing to sync, and returns an empty commits backlog, making the no-op sync successful — matching the behavior of Hudi and Parquet sources. ## How was this patch tested? - `./mvnw -pl xtable-core -am -DskipTests compile` - Existing `ITDeltaKernelConversionSource` / `TestDeltaKernelSync` suites cover incremental sync (local run pending CI). -- 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]
