MarigWeizhi commented on PR #3621:
URL: https://github.com/apache/amoro/pull/3621#issuecomment-2979057982

   > > 1. I've tested locally and just need to modify one place: 
`avgHealthScore`, which should return 100 when partitionCount is 0.
   > > 2. I'm not sure if it's necessary to change `targetSize` to 
`minTargetSize`
   > 
   > 1、I changed `targetSize` to `minTargetSize` according to the computational 
formula of TablePenaltyFactor in #3180 . I think there must have been a typing 
error before. 2、I also changed the computational formula of averageDataFileSize 
according to the description provided in #3180 . In the original code's 
calculation formula, the value of averageDataFileSize was incorrectly set to be 
less than or equal to 1. I made the necessary adjustments.
   
   LGTM


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