From 33e67a5391f84aa9b74961d06f28d40de9c27204 Mon Sep 17 00:00:00 2001
From: chenhang <chenhang@apache.org>
Date: Tue, 28 Feb 2023 10:58:56 +0800
Subject: [PATCH] Revert "upgrade jdk17. (#122)"

This reverts commit b299c403b89f12416438e7576838ae548c1645c4.
---
 .github/workflows/pr-unit-tests.yml |  9 +++------
 pom.xml                             | 24 ++++++++++--------------
 2 files changed, 13 insertions(+), 20 deletions(-)

diff --git a/.github/workflows/pr-unit-tests.yml b/.github/workflows/pr-unit-tests.yml
index d34d723..c3079ba 100644
--- a/.github/workflows/pr-unit-tests.yml
+++ b/.github/workflows/pr-unit-tests.yml
@@ -17,13 +17,10 @@ jobs:
 
     steps:
       - uses: actions/checkout@v1
-
-      - name: Set up JDK 17
-        uses: actions/setup-java@v2
-        if: ${{ needs.changed_files_job.outputs.docs_only != 'true' }}
+      - name: Set up JDK 1.8
+        uses: actions/setup-java@v1
         with:
-          distribution: 'temurin'
-          java-version: 17
+          java-version: 1.8
 
       - name: License check
         run: mvn -ntp -B license:check
diff --git a/pom.xml b/pom.xml
index d2ebe26..d680ed1 100644
--- a/pom.xml
+++ b/pom.xml
@@ -37,13 +37,13 @@
   <description>It is a lakehouse connector streaming convert data between lakehouse and Apache Pulsar.</description>
 
   <properties>
-    <maven.compiler.source>8</maven.compiler.source>
-    <maven.compiler.target>8</maven.compiler.target>
-    <java.version>17</java.version>
+    <maven.compiler.source>1.8</maven.compiler.source>
+    <maven.compiler.target>1.8</maven.compiler.target>
+    <java.version>1.8</java.version>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     <project.reporting.outputEncoding>UTF-8</project.reporting.outputEncoding>
     <redirectTestOutputToFile>true</redirectTestOutputToFile>
-    <spotbugs-annotations.version>4.2.2</spotbugs-annotations.version>
+    <spotbugs-annotations.version>3.1.8</spotbugs-annotations.version>
     <testRetryCount>0</testRetryCount>
 
     <!-- connector dependencies -->
@@ -71,11 +71,11 @@
     <!-- build plugin dependencies -->
     <license.plugin.version>3.0</license.plugin.version>
     <maven-checkstyle-plugin.version>3.1.2</maven-checkstyle-plugin.version>
-    <maven-compiler-plugin.version>3.10.1</maven-compiler-plugin.version>
-    <surefire.version>3.0.0-M3</surefire.version>
+    <maven-compiler-plugin.version>3.7.0</maven-compiler-plugin.version>
+    <maven-surefire-plugin.version>2.21.0</maven-surefire-plugin.version>
     <nifi.nar.plugin.version>1.2.0</nifi.nar.plugin.version>
     <puppycrawl.checkstyle.version>8.37</puppycrawl.checkstyle.version>
-    <spotbugs-maven-plugin.version>4.2.2</spotbugs-maven-plugin.version>
+    <spotbugs-maven-plugin.version>3.1.8</spotbugs-maven-plugin.version>
     <cloudPkgSuffix></cloudPkgSuffix>
   </properties>
 
@@ -564,7 +564,8 @@
           <artifactId>maven-compiler-plugin</artifactId>
           <version>${maven-compiler-plugin.version}</version>
           <configuration>
-            <encoding>UTF-8</encoding>
+            <source>${java.version}</source>
+            <target>${java.version}</target>
             <compilerArgs>
               <compilerArg>-Xlint:deprecation</compilerArg>
               <compilerArg>-Xlint:unchecked</compilerArg>
@@ -577,17 +578,12 @@
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-plugin</artifactId>
+          <version>${maven-surefire-plugin.version}</version>
           <configuration>
             <redirectTestOutputToFile>${redirectTestOutputToFile}</redirectTestOutputToFile>
             <reuseForks>false</reuseForks>
             <forkedProcessTimeoutInSeconds>1800</forkedProcessTimeoutInSeconds>
             <rerunFailingTestsCount>${testRetryCount}</rerunFailingTestsCount>
-            <argLine>
-              --add-opens java.base/java.util=ALL-UNNAMED
-              --add-opens java.base/java.lang=ALL-UNNAMED
-              --add-opens java.base/java.util.concurrent=ALL-UNNAMED
-              --add-opens java.base/java.nio=ALL-UNNAMED
-            </argLine>
           </configuration>
         </plugin>
         <!-- package -->