Repository: calcite Updated Branches: refs/heads/master 00c36467d -> d624a56ef
[CALCITE-2518] Add failOnWarnings to maven-javadoc-plugin configuration Missing tags like @param, @return are excluded from warnings list. Project: http://git-wip-us.apache.org/repos/asf/calcite/repo Commit: http://git-wip-us.apache.org/repos/asf/calcite/commit/d624a56e Tree: http://git-wip-us.apache.org/repos/asf/calcite/tree/d624a56e Diff: http://git-wip-us.apache.org/repos/asf/calcite/diff/d624a56e Branch: refs/heads/master Commit: d624a56ef987fbe79aa9a9076d68e7eb5bda0afb Parents: 00c3646 Author: Vladimir Sitnikov <[email protected]> Authored: Wed Sep 19 22:17:18 2018 +0300 Committer: Vladimir Sitnikov <[email protected]> Committed: Sun Sep 23 15:07:23 2018 +0300 ---------------------------------------------------------------------- .../main/java/org/apache/calcite/plan/volcano/RuleQueue.java | 2 +- core/src/main/java/org/apache/calcite/util/Bug.java | 7 ------- pom.xml | 4 ++++ 3 files changed, 5 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/calcite/blob/d624a56e/core/src/main/java/org/apache/calcite/plan/volcano/RuleQueue.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/calcite/plan/volcano/RuleQueue.java b/core/src/main/java/org/apache/calcite/plan/volcano/RuleQueue.java index 13dd779..45c4683 100644 --- a/core/src/main/java/org/apache/calcite/plan/volcano/RuleQueue.java +++ b/core/src/main/java/org/apache/calcite/plan/volcano/RuleQueue.java @@ -372,7 +372,7 @@ class RuleQueue { * <p>where W<sub>n, p</sub>, the weight of n within its parent p, is * * <blockquote>W<sub>n, p</sub> = Cost<sub>n</sub> / (SelfCost<sub>p</sub> + - * Cost<sub>n<sub>0</sub></sub> + ... + Cost<sub>n<sub>k</sub></sub>) + * Cost<sub>n0</sub> + ... + Cost<sub>nk</sub>) * </blockquote> */ double computeImportance(RelSubset subset) { http://git-wip-us.apache.org/repos/asf/calcite/blob/d624a56e/core/src/main/java/org/apache/calcite/util/Bug.java ---------------------------------------------------------------------- diff --git a/core/src/main/java/org/apache/calcite/util/Bug.java b/core/src/main/java/org/apache/calcite/util/Bug.java index 7eeb208..7930dbc 100644 --- a/core/src/main/java/org/apache/calcite/util/Bug.java +++ b/core/src/main/java/org/apache/calcite/util/Bug.java @@ -40,13 +40,6 @@ package org.apache.calcite.util; * the fix is integrated into other branches, the constant will be removed from * those branches.</p> * - * <p><b>To do</b></p> - * - * <p>The following is a list of tasks to be completed before committing to - * master branch.</p> - * - * <ul> - * </ul> */ public abstract class Bug { //~ Static fields/initializers --------------------------------------------- http://git-wip-us.apache.org/repos/asf/calcite/blob/d624a56e/pom.xml ---------------------------------------------------------------------- diff --git a/pom.xml b/pom.xml index f90b495..5fe887b 100644 --- a/pom.xml +++ b/pom.xml @@ -745,10 +745,14 @@ limitations under the License. <artifactId>maven-javadoc-plugin</artifactId> <configuration> <additionalparam>-html5</additionalparam> + <doclint>all,-missing</doclint> + <failOnWarnings>true</failOnWarnings> <links> <link>${maven-javadoc-plugin.link}</link> </links> + <notimestamp>true</notimestamp> <excludePackageNames>${maven-javadoc-plugin.excludePackageNames}</excludePackageNames> + <quiet>true</quiet> <show>private</show> </configuration> </plugin>
