Repository navigation
Updates Delta CDC read to fail when schemas... (#40868) - #40877
Conversation
There was a problem hiding this comment.
📋 Review Summary
This PR cleanly cherry-picks #40868 to the release-2.78 branch to fix #40853. It introduces DeltaSchemaUtils to validate schema read-compatibility across Delta table versions during CDC reads, ensuring only additive schema changes are permitted between the start and end versions.
🔍 General Feedback
- The cherry-pick matches the original commit in #40868 with no discrepancies.
- Schema validation is properly applied both early during pipeline expansion (
DeltaIO.ReadChanges.expand) and across intermediate commits (METADATAactions) during task planning (CreateCDCReadTasksDoFn). - Unit and integration tests thoroughly cover additive changes, metadata-only changes, column mapping, and rejected non-additive evolutions.
💡 Tip: Comment /gemini review on this pull request to trigger a new Gemini code review.
|
Assigning reviewers: R: @ahmedabu98 for label delta. Note: If you would like to opt out of this review, comment Available commands:
The PR bot will only process comments in the main thread (not review comments). |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## release-2.78 #40877 +/- ##
===============================================
Coverage ? 0
===============================================
Files ? 0
Lines ? 0
Branches ? 0
===============================================
Hits ? 0
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
...are different at start and end.
Cherry-picks #40868 to the 2.78.0 release branch.
This fixes #40853.