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

   ### Change Logs
   
   Added infer logic to `SevenToEightUpgradeHandler` similar to 
`HoodieTableMetaClient.inferRecordMergeMode()`. Record merge mode is required 
to dictate the merging behavior in release 1.x after merging 
https://github.com/apache/hudi/pull/9894. It plays the same role as the payload 
class config in the release 0.x.
   
   As mentioned in `HoodieTableVersion`:
   - version 7 is Hudi 0.16, which has not been released yet,
   - version 8 is Hudi 1.0 beta.
   
   So I couldn't make end-to-end tests with upgrade from 7 to 8.
   And due to added completion time in the timeline after 
https://github.com/apache/hudi/pull/9617, I couldn't update and read some table 
by Hudi 1.0 beta, after it was write by Hudi 0.14 (version 6).
   
   I could only write some table by version 6, and try to call upgrade with 
checking `hoodie.properties`. I did this check locally by using this scripts:
   https://github.com/geserdugarov/test-hudi-issues/tree/main/HUDI-7847
   
   ### Impact
   
   Low
   
   ### Risk level (write none, low medium or high below)
   
   Low
   
   ### Documentation Update
   
   No need
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Change Logs and Impact were stated clearly
   - [x] 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