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

   ### Describe the issue this Pull Request addresses
   
   For pre-v9 tables (e.g., version 6), getRecordMergeMode() returns null 
because the hoodie.record.merge.mode property didn't exist. This caused 
generateRequiredSchema to skip the CUSTOM merge mode path, returning selective 
fields instead of the full table schema for custom payload classes like 
HoodieMetadataPayload.                                                          
                   
                                                                                
                                     
   The fix adds the same inferMergingConfigsForPreV9Table inference that 
getMandatoryFieldsForMerging already uses, ensuring consistent behavior between 
v6 and v9 tables.
   
   ### Impact
   
   No public API changes. For pre-v9 tables using custom payload classes (e.g., 
metadata tables with  HoodieMetadataPayload), the file group reader will now 
correctly return the full table schema instead of  selective fields, matching 
the behavior of v9 tables.
   
   ### Risk Level
   
   Low
   
   ### Documentation Update
   
   None. This is an internal bug fix with no new configs or user-facing changes.
   
   ### Contributor's checklist
   
   - [ ] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [ ] Enough context is provided in the sections above
   - [ ] 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