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

dockerzhang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-inlong.git


The following commit(s) were added to refs/heads/master by this push:
     new b5ed34895 [INLONG-3716][Manager] Optimize the manager plugin 
dependency (#3717)
b5ed34895 is described below

commit b5ed34895d422fe2a02a0d89184778b1488220a5
Author: dockerzhang <[email protected]>
AuthorDate: Fri Apr 15 09:27:48 2022 +0800

    [INLONG-3716][Manager] Optimize the manager plugin dependency (#3717)
---
 inlong-audit/bin/proxy-start.sh                    |  2 +-
 inlong-dataproxy/bin/dataproxy-start.sh            |  2 +-
 inlong-manager/manager-plugins/pom.xml             | 59 ----------------------
 .../manager/plugin/flink/ManagerFlinkTask.java     |  1 +
 .../inlong/manager/plugin/util/FlinkUtils.java     |  5 +-
 .../service/source/SourceSnapshotOperation.java    |  8 ++-
 inlong-manager/manager-web/pom.xml                 |  5 ++
 pom.xml                                            |  5 --
 8 files changed, 17 insertions(+), 70 deletions(-)

diff --git a/inlong-audit/bin/proxy-start.sh b/inlong-audit/bin/proxy-start.sh
index b3bc3d0a7..3cb1c1295 100644
--- a/inlong-audit/bin/proxy-start.sh
+++ b/inlong-audit/bin/proxy-start.sh
@@ -54,7 +54,7 @@ if [ ! -d "${LOG_DIR}" ]; then
   mkdir ${LOG_DIR}
 fi
 if [ -f "$CONFIG_FILE_WITH_PATH" ]; then
-  nohup bin/audit-proxy agent --conf conf/ -f 
"${CONFIG_FILE_WITH_COFING_PATH}" -n agent1 --no-reload-conf 
1>${LOG_DIR}/proxy.log 2>${LOG_DIR}/proxy-error.log &
+  nohup bash +x bin/audit-proxy agent --conf conf/ -f 
"${CONFIG_FILE_WITH_COFING_PATH}" -n agent1 --no-reload-conf 
1>${LOG_DIR}/proxy.log 2>${LOG_DIR}/proxy-error.log &
 else
   error "${CONFIG_FILE_WITH_PATH} is not exist! start failed!" 1
 fi
diff --git a/inlong-dataproxy/bin/dataproxy-start.sh 
b/inlong-dataproxy/bin/dataproxy-start.sh
index 71b582923..fcf02a9d1 100755
--- a/inlong-dataproxy/bin/dataproxy-start.sh
+++ b/inlong-dataproxy/bin/dataproxy-start.sh
@@ -51,7 +51,7 @@ CONFIG_FILE_WITH_COFING_PATH="conf/${CONFIG_FILE}"
 CONFIG_FILE_WITH_PATH="${basedir}/${CONFIG_FILE}"
 
 if [ -f "$CONFIG_FILE_WITH_PATH" ]; then
-  nohup bin/dataproxy-ng agent --conf conf/ -f 
"${CONFIG_FILE_WITH_COFING_PATH}" -n agent1 --no-reload-conf  > /dev/null 2>&1 &
+  nohup bash +x bin/dataproxy-ng agent --conf conf/ -f 
"${CONFIG_FILE_WITH_COFING_PATH}" -n agent1 --no-reload-conf  > /dev/null 2>&1 &
 else
    error "${CONFIG_FILE_WITH_PATH} is not exist! start failed!" 1
 fi
\ No newline at end of file
diff --git a/inlong-manager/manager-plugins/pom.xml 
b/inlong-manager/manager-plugins/pom.xml
index 935984174..6bf990212 100644
--- a/inlong-manager/manager-plugins/pom.xml
+++ b/inlong-manager/manager-plugins/pom.xml
@@ -51,10 +51,6 @@
             <groupId>org.apache.flink</groupId>
             <artifactId>flink-clients_2.11</artifactId>
         </dependency>
-        <dependency>
-            <groupId>org.apache.flink</groupId>
-            <artifactId>flink-core</artifactId>
-        </dependency>
         <dependency>
             <groupId>com.fasterxml.jackson.core</groupId>
             <artifactId>jackson-core</artifactId>
@@ -67,62 +63,7 @@
     </dependencies>
 
     <build>
-        <resources>
-            <resource>
-                <directory>src/main/resources</directory>
-                <includes>
-                    <include>META-INF/*</include>
-                    <include>flink-sort-plugin.properties</include>
-                </includes>
-            </resource>
-        </resources>
-        <finalName>manager-plugins</finalName>
         <plugins>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-compiler-plugin</artifactId>
-                <configuration>
-                    <encoding>${project.build.encoding}</encoding>
-                    <source>${maven.compiler.source}</source>
-                    <target>${maven.compiler.target}</target>
-                    <annotationProcessorPaths>
-                        <path>
-                            <groupId>org.projectlombok</groupId>
-                            <artifactId>lombok</artifactId>
-                            <version>${lombok.version}</version>
-                        </path>
-                    </annotationProcessorPaths>
-                </configuration>
-                <version>${plugin.compile.version}</version>
-            </plugin>
-            <plugin>
-                <groupId>org.apache.maven.plugins</groupId>
-                <artifactId>maven-shade-plugin</artifactId>
-                <version>3.2.4</version>
-                <executions>
-                    <execution>
-                        <phase>package</phase>
-                        <goals>
-                            <goal>shade</goal>
-                        </goals>
-                        <configuration>
-                            
<createDependencyReducedPom>false</createDependencyReducedPom>
-                            
<shadedArtifactAttached>false</shadedArtifactAttached>
-                            
<finalName>${project.artifactId}-${project.version}</finalName>
-                            <filters>>
-                                <filter>
-                                    <artifact>*</artifact>
-                                    <excludes>
-                                        <exclude>META-INF/*.SF</exclude>
-                                        <exclude>META-INF/*.DSA</exclude>
-                                        <exclude>META-INF/*.RSA</exclude>
-                                    </excludes>
-                                </filter>
-                            </filters>
-                        </configuration>
-                    </execution>
-                </executions>
-            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-resources-plugin</artifactId>
diff --git 
a/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/flink/ManagerFlinkTask.java
 
b/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/flink/ManagerFlinkTask.java
index 1ddbe1523..4b9fcdc69 100644
--- 
a/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/flink/ManagerFlinkTask.java
+++ 
b/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/flink/ManagerFlinkTask.java
@@ -88,6 +88,7 @@ public class ManagerFlinkTask {
     }
 
     /**
+     * genPath
      * @param flinkInfo
      * @param dataflow
      */
diff --git 
a/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/util/FlinkUtils.java
 
b/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/util/FlinkUtils.java
index 7b0384e44..7e3654fd8 100644
--- 
a/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/util/FlinkUtils.java
+++ 
b/inlong-manager/manager-plugins/src/main/java/org/apache/inlong/manager/plugin/util/FlinkUtils.java
@@ -38,11 +38,10 @@ import java.util.regex.Pattern;
 public class FlinkUtils {
     public static final String BASE_DIRECTORY = "config";
 
-    /**
-     */
     public static final List<String> FLINK_VERSION_COLLECTION = 
Arrays.asList("Flink-1.13");
 
     /**
+     * getLatestFlinkVersion
      * @param supportedFlink
      * @return
      */
@@ -116,6 +115,7 @@ public class FlinkUtils {
     }
 
     /**
+     * getConfigDirectory
      * @param name
      * @return
      */
@@ -124,6 +124,7 @@ public class FlinkUtils {
     }
 
     /**
+     * writeConfigToFile
      * @param configJobDirectory
      * @param configFileName
      * @param content
diff --git 
a/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/SourceSnapshotOperation.java
 
b/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/SourceSnapshotOperation.java
index 265650ed2..f850b7719 100644
--- 
a/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/SourceSnapshotOperation.java
+++ 
b/inlong-manager/manager-service/src/main/java/org/apache/inlong/manager/service/source/SourceSnapshotOperation.java
@@ -116,10 +116,14 @@ public class SourceSnapshotOperation implements 
AutoCloseable {
         String agentIp = request.getAgentIp();
         List<TaskSnapshotMessage> snapshotList = request.getSnapshotList();
         if (CollectionUtils.isEmpty(snapshotList)) {
-            LOGGER.info("receive snapshot from ip={}, but snapshot list is 
empty", agentIp);
+            if (LOGGER.isDebugEnabled()) {
+                LOGGER.info("receive snapshot from ip={}, but snapshot list is 
empty", agentIp);
+            }
             return true;
         }
-        LOGGER.debug("receive snapshot from ip={}, msg size={}", agentIp, 
snapshotList.size());
+        if (LOGGER.isDebugEnabled()) {
+            LOGGER.debug("receive snapshot from ip={}, msg size={}", agentIp, 
snapshotList.size());
+        }
 
         try {
             // Offer the request of snapshot to the queue, and another thread 
will parse the data in the queue.
diff --git a/inlong-manager/manager-web/pom.xml 
b/inlong-manager/manager-web/pom.xml
index 602bd1f23..7dbca030d 100644
--- a/inlong-manager/manager-web/pom.xml
+++ b/inlong-manager/manager-web/pom.xml
@@ -46,6 +46,11 @@
             <artifactId>manager-client-tools</artifactId>
             <version>${project.version}</version>
         </dependency>
+        <dependency>
+            <groupId>org.apache.inlong</groupId>
+            <artifactId>manager-plugins</artifactId>
+            <version>${project.version}</version>
+        </dependency>
         <dependency>
             <groupId>org.springframework.boot</groupId>
             <artifactId>spring-boot-starter-web</artifactId>
diff --git a/pom.xml b/pom.xml
index c42a6201c..e5f893603 100644
--- a/pom.xml
+++ b/pom.xml
@@ -580,11 +580,6 @@
                 <artifactId>spring-core</artifactId>
                 <version>${spring.version}</version>
             </dependency>
-            <dependency>
-                <groupId>org.springframework</groupId>
-                <artifactId>spring-context</artifactId>
-                <version>${spring.version}</version>
-            </dependency>
             <dependency>
                 <groupId>org.springframework</groupId>
                 <artifactId>spring-jdbc</artifactId>

Reply via email to