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

   ## What is the purpose of the pull request
   
   When the `instant_time.rollback.requested` file in the timeline is empty or 
corrupted, it cannot be parsed.  When running `getPendingRollbackInfos()`, it's 
going to skip that empty/corrupted requested rollback instant and the rollback 
instant is going to stay on the timeline forever, preventing metadata table 
archival.
   
   This PR fixes the problem by deleting the requested rollback plan if it is 
empty or corrupted.
   
   ## Brief change log
   
     - Adds logic to delete requested rollback plan if it cannot be parsed in 
`BaseHoodieWriteClient::getPendingRollbackInfos()`.
   
   ## Verify this pull request
   
   This change added tests for requested rollbacks, either valid or corrupted, 
in `TestClientRollback`.  The fix is also verified by running the deltastreamer 
on a Hudi Table with corrupted requested rollback in the timeline.  The 
corrupted rollback plan is deleted afterwards.
   
   ## Committer checklist
   
    - [ ] Has a corresponding JIRA in PR title & commit
    
    - [ ] Commit message is descriptive of the change
    
    - [ ] CI is green
   
    - [ ] Necessary doc changes done or have another open PR
          
    - [ ] For large changes, please consider breaking it into sub-tasks under 
an umbrella JIRA.
   


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