You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by "danny0405 (via GitHub)" <gi...@apache.org> on 2023/04/15 07:08:38 UTC

[GitHub] [hudi] danny0405 commented on pull request #8455: [HUDI-6076] Change clustering.plan.strategy.small.file.limit's unit to byte

danny0405 commented on PR #8455:
URL: https://github.com/apache/hudi/pull/8455#issuecomment-1509602377

   > @Zouxxyy, did you take the compatibility of this change into consideration? With this change, the config value of `clustering.plan.strategy.small.file.limit` must be changed when upgrade to the lastest version. IMO, you should introduce new option for the config unification.
   
   A valid concern, this may break the user code, we should take the modification as early and add some illustrations on the release note.


-- 
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-unsubscribe@hudi.apache.org

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