yihua opened a new pull request, #19943:
URL: https://github.com/apache/hudi/pull/19943

   ### Describe the issue this Pull Request addresses
   
   closes #19942
   
   ### Summary and Changelog
   
   `ErrorTableAwareChainedTransformer` added `_corrupt_record` once and then 
validated after every transformer, throwing `HoodieValidationException` if the 
column was gone. Any transformer that projects a subset of columns removes it, 
so a custom transformer doing `dataset.select(userColumns)` failed the sync on 
a column the user never asked about.
   
   - Re-inject the column after each transformer instead of validating. A 
transformer that keeps it sees a no-op; one that drops it gets it back with 
null values, the same state the chain establishes before the first transformer.
   - `ErrorTableUtils.validate` had no other caller and is removed.
   - `TestErrorTableAwareChainedTransformer.testForErrorRecordColumn` asserted 
the throw; it is replaced by three tests covering a dropping transformer 
mid-chain, a single projecting transformer, and a transformer that preserves 
the column.
   
   ### Impact
   
   Error-table users running a column-projecting transformer stop failing. No 
config or API change.
   
   ### Risk Level
   
   low. The re-injection reuses the helper the chain already calls before the 
first transformer, and is a no-op when the column is present.
   
   ### Documentation Update
   
   none
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Enough context is provided in the sections above
   - [x] Adequate tests were added if applicable
   


-- 
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]

Reply via email to