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

   ### Change Logs
   
   This is a follow-up for #9871 . We can eliminate the restrictions for 2 
reasons:
   
   1. we now always generate fresh new compaction instant timestamp instead of 
concatenating with a special suffix, so the filtering and ordering based on the 
instant time is now broken(even under OCC, the compaction instant time would be 
greater than the instant from DT that triggers it.) The MDT reader actually 
checks the validity for the instants from MDT after the fix: 
https://github.com/apache/hudi/pull/8797
   2. the rollback and cleaning for MOR table log files is simplified just as 
the COW table to remove log files, so the rollback is very straightforward, and 
there is no need for this check.
   
   ### Impact
   
   none
   
   ### Risk level (write none, low medium or high below)
   
   low medium
   
   ### Documentation Update
   
   none
   
   ### 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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to