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 <[email protected]>
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 "
