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

   ### Change Logs
   
   LogFileComparator does not compare cdc suffix, for example, for the 
following logs, they will be considered equal:
   
   ```text
   .file1_2.log.1_1-1-1
   .file1_2.log.1_1-1-1.cdc
   ```
   
   One problem it will bring is: because `new 
TreeSet<>(HoodieLogFile.getReverseLogFileComparator())` is used when 
constructing FG, when the two logs are exactly the same except for suffix, only 
one of them will exist in logFiles.
   
   ### Impact
   
   Fix LogFileComparator
   
   ### Risk level (write none, low medium or high below)
   
   low
   
   ### 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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to