You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by va...@apache.org on 2020/10/17 08:01:57 UTC

[camel-kafka-connector] 01/03: Catalog Order connectors in descriptor generation.

This is an automated email from the ASF dual-hosted git repository.

valdar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel-kafka-connector.git

commit 0a0961749b6a0445d22477e966882bad51a755dd
Author: Andrea Tarocchi <an...@gmail.com>
AuthorDate: Sat Oct 17 08:25:03 2020 +0200

    Catalog Order connectors in descriptor generation.
---
 .../descriptor/CollectConnectorDescriptorMojo.java | 35 ++++++++--------------
 .../maven/CamelKafkaConnectorUpdateMojo.java       |  3 ++
 2 files changed, 15 insertions(+), 23 deletions(-)

diff --git a/tooling/camel-kafka-connector-catalog-descriptor-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/catalog/descriptor/CollectConnectorDescriptorMojo.java b/tooling/camel-kafka-connector-catalog-descriptor-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/catalog/descriptor/CollectConnectorDescriptorMojo.java
index 4049e41..ce4d87e 100644
--- a/tooling/camel-kafka-connector-catalog-descriptor-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/catalog/descriptor/CollectConnectorDescriptorMojo.java
+++ b/tooling/camel-kafka-connector-catalog-descriptor-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/catalog/descriptor/CollectConnectorDescriptorMojo.java
@@ -23,11 +23,11 @@ import java.io.FileInputStream;
 import java.io.FileWriter;
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.util.Arrays;
+import java.util.stream.Stream;
 
 import org.apache.commons.io.FileUtils;
 import org.apache.maven.plugin.AbstractMojo;
-import org.apache.maven.plugin.MojoExecutionException;
-import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.plugins.annotations.Mojo;
 import org.apache.maven.plugins.annotations.Parameter;
 import org.apache.maven.project.MavenProject;
@@ -35,7 +35,7 @@ import org.apache.maven.project.MavenProject;
 /**
  * Updates connector.properties file
  */
-@Mojo(name = "list-descriptor-files", threadSafe = true)
+@Mojo(name = "update-descriptor-files", threadSafe = true)
 public class CollectConnectorDescriptorMojo extends AbstractMojo {
 
     /**
@@ -53,41 +53,29 @@ public class CollectConnectorDescriptorMojo extends AbstractMojo {
     /**
      * The directory for catalog descriptors
      */
-    @Parameter(defaultValue = "${project.directory}/../../camel-kafka-connector-catalog/src/generated/resources/descriptors")
+    @Parameter(defaultValue = "${project.directory}/../src/generated/resources/descriptors")
     protected File catalogDescriptorDir;
 
     /**
-     * The connectors project name parameter.
-     */
-    @Parameter(property = "connectors-project-name", defaultValue = "connectors", readonly = true)
-    protected String connectorsProjectName;
-
-    /**
      * Execute goal.
-     *
-     * @throws MojoExecutionException execution of the main class or one of the
-     *             threads it generated failed.
-     * @throws MojoFailureException something bad happened...
      */
     @Override
-    public void execute() throws MojoExecutionException, MojoFailureException {
-        if (!project.getArtifactId().equals(connectorsProjectName)) {
-            getLog()
-                .debug("Skipping project " + project.getArtifactId() + " since it is not " + connectorsProjectName + " can be configured with <connectors-project-name> option.");
-            return;
-        }
+    public void execute() {
         try {
             executeComponentsReadme();
         } catch (IOException e) {
-            e.printStackTrace();
+            getLog().error("An error occurred while running camel-kakfa-connector-descriptor-maven-plugin:update-descriptor-files", e);
         }
     }
 
-    protected void executeComponentsReadme() throws MojoExecutionException, MojoFailureException, IOException {
-
+    protected void executeComponentsReadme() throws IOException {
+        getLog().info("About to scan all projects in" + connectorsDir + " in order to gather descriptors files");
         if (connectorsDir != null && connectorsDir.isDirectory()) {
             File[] files = connectorsDir.listFiles();
+            getLog().info("Found " + (files != null ? String.valueOf(files.length) : "0") + "directories:");
             if (files != null) {
+                Arrays.sort(files);
+                Stream.of(files).filter(f -> f.isDirectory()).forEach(f -> getLog().info(f.getPath()));
                 StringBuilder sb = new StringBuilder();
                 for (int i = 0; i < files.length; i++) {
                     File file = files[i];
@@ -119,6 +107,7 @@ public class CollectConnectorDescriptorMojo extends AbstractMojo {
                     }
                 }
                 File file = FileUtils.getFile(catalogDescriptorDir, "connectors.properties");
+                getLog().info("About to write descriptor to: " + catalogDescriptorDir);
                 file.createNewFile();
                 try (BufferedWriter writer = new BufferedWriter(new FileWriter(file))) {
                     writer.write(sb.toString());
diff --git a/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java b/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
index 03ca7ee..e93b3d0 100644
--- a/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
+++ b/tooling/camel-kafka-connector-generator-maven-plugin/src/main/java/org/apache/camel/kafkaconnector/maven/CamelKafkaConnectorUpdateMojo.java
@@ -591,7 +591,10 @@ public class CamelKafkaConnectorUpdateMojo extends AbstractCamelKafkaConnectorMo
         } else {
             getLog().debug("No changes to website doc file: " + docFileWebsite);
         }
+
+        // generate json descriptor src/generated/resources/<connector-name>.json
         writeJson(listOptions, getMainDepArtifactId(), connectorDir, ct, packageName + "." + javaClassConnectorName, convertersList, transformsList, aggregationStrategiesList);
+        // generate descriptor src/generated/descriptors/connector-{sink,source}.properties
         writeDescriptors(connectorDir, ct);
     }