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

   ### Change Logs
   
   De-coupling column drop flag and schema validation flag. Looks like we had 
tight coupling before. 
   for eg, 
   if table schema is col1, col2, col3
   and new incoming schema is col1, col2
   col drop config is set to false (which means col drop should not be 
supported), and schema validation is set to false, commit will succeed. 
Expectation is, commit should fail for this new batch. 
   
   Fixing the use-case in this patch, where we are de-coupling them. col drop 
flag will be honored irrespective of whether schema validation is enabled or 
not. 
   
   ### Impact
   
   Column drop flag will be honored in all cases. 
   
   ### Risk level (write none, low medium or high below)
   
   low.
   
   ### Documentation Update
   
   N/A
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Change Logs and Impact were stated clearly
   - [ ] Adequate tests were added if applicable
   - [ ] CI passed
   


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