This is an automated email from the ASF dual-hosted git repository. leesf pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-hudi.git
The following commit(s) were added to refs/heads/master by this push: new 1713f68 [MINOR] Add error message when check arguments (#1451) 1713f68 is described below commit 1713f686f86e8c2f0a908c313cca9b595c6aed33 Author: Raymond Xu <2701446+xushi...@users.noreply.github.com> AuthorDate: Thu Mar 26 19:21:38 2020 -0700 [MINOR] Add error message when check arguments (#1451) --- .../main/java/org/apache/hudi/config/HoodieCompactionConfig.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/hudi-client/src/main/java/org/apache/hudi/config/HoodieCompactionConfig.java b/hudi-client/src/main/java/org/apache/hudi/config/HoodieCompactionConfig.java index 074ea78..a7275c8 100644 --- a/hudi-client/src/main/java/org/apache/hudi/config/HoodieCompactionConfig.java +++ b/hudi-client/src/main/java/org/apache/hudi/config/HoodieCompactionConfig.java @@ -288,7 +288,11 @@ public class HoodieCompactionConfig extends DefaultHoodieConfig { int maxInstantsToKeep = Integer.parseInt(props.getProperty(HoodieCompactionConfig.MAX_COMMITS_TO_KEEP_PROP)); int cleanerCommitsRetained = Integer.parseInt(props.getProperty(HoodieCompactionConfig.CLEANER_COMMITS_RETAINED_PROP)); - ValidationUtils.checkArgument(maxInstantsToKeep > minInstantsToKeep); + ValidationUtils.checkArgument(maxInstantsToKeep > minInstantsToKeep, + String.format( + "Increase %s=%d to be greater than %s=%d.", + HoodieCompactionConfig.MAX_COMMITS_TO_KEEP_PROP, maxInstantsToKeep, + HoodieCompactionConfig.MIN_COMMITS_TO_KEEP_PROP, minInstantsToKeep)); ValidationUtils.checkArgument(minInstantsToKeep > cleanerCommitsRetained, String.format( "Increase %s=%d to be greater than %s=%d. Otherwise, there is risk of incremental pull "