diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 19659f2d79..f81c1ad8b1 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -27,7 +27,7 @@ jobs:
build:
strategy:
matrix:
- java-version: [8, 11]
+ java-version: [8, 11, 17]
os: [ubuntu-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
diff --git a/tez-tools/pom.xml b/tez-tools/pom.xml
index 8811d80687..b5a8326c12 100644
--- a/tez-tools/pom.xml
+++ b/tez-tools/pom.xml
@@ -50,6 +50,25 @@
+ * http://www.apache.org/licenses/LICENSE-2.0 + *
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -21,6 +21,15 @@
import java.io.IOException;
import java.util.Map;
+import com.sun.javadoc.AnnotationDesc;
+import com.sun.javadoc.AnnotationDesc.ElementValuePair;
+import com.sun.javadoc.ClassDoc;
+import com.sun.javadoc.DocErrorReporter;
+import com.sun.javadoc.FieldDoc;
+import com.sun.javadoc.LanguageVersion;
+import com.sun.javadoc.RootDoc;
+import com.sun.tools.doclets.standard.Standard;
+
import org.apache.hadoop.classification.InterfaceAudience.Private;
import org.apache.hadoop.classification.InterfaceStability.Evolving;
import org.apache.hadoop.classification.InterfaceStability.Unstable;
@@ -31,21 +40,13 @@
import org.apache.tez.tools.javadoc.util.HtmlWriter;
import org.apache.tez.tools.javadoc.util.XmlWriter;
-import com.sun.javadoc.AnnotationDesc;
-import com.sun.javadoc.AnnotationDesc.ElementValuePair;
-import com.sun.javadoc.ClassDoc;
-import com.sun.javadoc.DocErrorReporter;
-import com.sun.javadoc.FieldDoc;
-import com.sun.javadoc.LanguageVersion;
-import com.sun.javadoc.RootDoc;
-import com.sun.tools.doclets.standard.Standard;
-
public final class ConfigStandardDoclet {
private static final String DEBUG_SWITCH = "-debug";
private static boolean debugMode = false;
- private ConfigStandardDoclet() {}
+ private ConfigStandardDoclet() {
+ }
public static LanguageVersion languageVersion() {
return LanguageVersion.JAVA_1_5;
@@ -109,7 +110,7 @@ private static void processDoc(ClassDoc doc) {
logMessage("Processing config class: " + doc);
Config config = new Config(doc.name(), templateName);
- Map
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.tez.tools.javadoc.model;
+
+import java.util.Map;
+import java.util.TreeMap;
+
+public class Config {
+
+ private final String templateName;
+ private final String configName;
+ private Map
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.tez.tools.javadoc.model;
+
+public class ConfigProperty {
+
+ private String propertyName;
+ private String defaultValue;
+ private String description;
+ private String type = "string";
+ private boolean isPrivate = false;
+ private boolean isUnstable = false;
+ private boolean isEvolving = false;
+ private boolean isValidConfigProp = false;
+ private String inferredType;
+
+ public String getPropertyName() {
+ return propertyName;
+ }
+
+ public void setPropertyName(String propertyName) {
+ this.propertyName = propertyName;
+ }
+
+ public String getDefaultValue() {
+ return defaultValue;
+ }
+
+ public void setDefaultValue(String defaultValue) {
+ this.defaultValue = defaultValue;
+ }
+
+ public String getDescription() {
+ return description;
+ }
+
+ public void setDescription(String description) {
+ this.description = description;
+ }
+
+ public String getType() {
+ return type;
+ }
+
+ public void setType(String type) {
+ this.type = type;
+ }
+
+ public boolean isPrivate() {
+ return isPrivate;
+ }
+
+ public void setPrivate(boolean aPrivate) {
+ isPrivate = aPrivate;
+ }
+
+ public boolean isUnstable() {
+ return isUnstable;
+ }
+
+ public void setUnstable(boolean unstable) {
+ isUnstable = unstable;
+ }
+
+ public boolean isEvolving() {
+ return isEvolving;
+ }
+
+ public void setEvolving(boolean evolving) {
+ isEvolving = evolving;
+ }
+
+ public boolean isValidConfigProp() {
+ return isValidConfigProp;
+ }
+
+ public void setValidConfigProp(boolean validConfigProp) {
+ isValidConfigProp = validConfigProp;
+ }
+
+ public String getInferredType() {
+ return inferredType;
+ }
+
+ public void setInferredType(String inferredType) {
+ this.inferredType = inferredType;
+ }
+
+ @Override
+ public String toString() {
+ return "name=" + getPropertyName()
+ + ", defaultValue=" + getDefaultValue()
+ + ", description=" + getDescription()
+ + ", type=" + getType()
+ + ", inferredType=" + getInferredType()
+ + ", private=" + isPrivate()
+ + ", isConfigProp=" + isValidConfigProp();
+ }
+}
diff --git a/tez-tools/tez-javadoc-tools/src/main/java/org/apache/tez/tools/javadoc/model/Config.java b/tez-tools/tez-javadoc-tools-base-jdk8/src/main/java/org/apache/tez/tools/javadoc/model/package-info.java
similarity index 65%
rename from tez-tools/tez-javadoc-tools/src/main/java/org/apache/tez/tools/javadoc/model/Config.java
rename to tez-tools/tez-javadoc-tools-base-jdk8/src/main/java/org/apache/tez/tools/javadoc/model/package-info.java
index 604d48ac5e..76acd05258 100644
--- a/tez-tools/tez-javadoc-tools/src/main/java/org/apache/tez/tools/javadoc/model/Config.java
+++ b/tez-tools/tez-javadoc-tools-base-jdk8/src/main/java/org/apache/tez/tools/javadoc/model/package-info.java
@@ -16,25 +16,7 @@
* limitations under the License.
*/
+@Private
package org.apache.tez.tools.javadoc.model;
-import java.util.Map;
-import java.util.TreeMap;
-
-public class Config {
-
- public final String templateName;
- public final String configName;
- public Map
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -19,13 +19,11 @@
package org.apache.tez.tools.javadoc.util;
import java.io.File;
-import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.OutputStreamWriter;
import java.io.PrintWriter;
-import org.apache.tez.dag.api.TezException;
import org.apache.tez.tools.javadoc.model.Config;
import org.apache.tez.tools.javadoc.model.ConfigProperty;
@@ -36,12 +34,12 @@ public class HtmlWriter extends Writer {
public void write(Config config) throws IOException {
PrintWriter out = null;
- if (config.configName == null || config.configName.isEmpty()) {
+ if (config.getConfigName() == null || config.getConfigName().isEmpty()) {
throw new RuntimeException("Config Name is null or empty");
}
try {
- File file = new File(config.configName + ".html");
+ File file = new File(config.getConfigName() + ".html");
out = new PrintWriter(new OutputStreamWriter(new FileOutputStream(file), "UTF-8"));
out.println("");
@@ -52,7 +50,7 @@ public void write(Config config) throws IOException {
out.println(""+ config.configName +"
");
+ out.println("" + config.getConfigName() + "
");
out.println("
");
out.println("");
@@ -118,29 +116,35 @@ public void write(Config config) throws IOException {
out.println("
" + "Is Evolving?" + " ");
out.println("");
- for (ConfigProperty configProperty : config.configProperties.values()) {
+ for (ConfigProperty configProperty : config.getConfigProperties().values()) {
if (!isValidConfigProperty(configProperty)) {
continue;
}
String altClass = "";
- if (configProperty.isPrivate) {
+ if (configProperty.isPrivate()) {
altClass = "class=\"tr_private\"";
- } else if (configProperty.isEvolving || configProperty.isUnstable) {
+ } else if (configProperty.isEvolving() || configProperty.isUnstable()) {
altClass = "class=\"tr_evolve_unstable\"";
}
out.println("");
- out.println(" ");
}
@@ -150,12 +154,10 @@ public void write(Config config) throws IOException {
out.println("");
out.println("" + configProperty.propertyName + " ");
- out.println("" + configProperty.defaultValue + " ");
- out.println("" + configProperty.description + " ");
- out.println("" + configProperty.type + " ");
+ out.println("" + configProperty.getPropertyName() + " ");
+ out.println("" + configProperty.getDefaultValue() + " ");
+ out.println("" + configProperty.getDescription() + " ");
+ out.println("" + configProperty.getType() + " ");
// Re-enable after adding values
// out.println("" + configProperty.validValues + " ");
- out.println("" + configProperty.isPrivate + " ");
- out.println("" + configProperty.isEvolving + " ");
- out.println("" + configProperty.isUnstable + " ");
+ out.println(
+ "" + configProperty.isPrivate() + " ");
+ out.println(
+ ""
+ + configProperty.isEvolving() + " ");
+ out.println(
+ ""
+ + configProperty.isUnstable() + " ");
out.println("