From 36f810e9750e00a229844e1f00aa5876b08cc645 Mon Sep 17 00:00:00 2001
From: mystic-lama <mysticlama000@gmail.com>
Date: Mon, 23 Oct 2023 21:34:29 -0700
Subject: [PATCH] ORC-1520: Removed JDK 8 settings from pom

### What changes were proposed in this pull request?
The PR proposes to remove jdk8 related parameters from pom, since we moved to JDK 17 as part of #1627

### Why are the changes needed?
Since we moved to JDK 17, JDK 8 parameters are not relevant any more.
This shall keep files up to date with changes

### How was this patch tested?
Build runs successfully with the changes.

Closes #1639 from mystic-lama/ORC-1520_remove_jdk8_settings.

Lead-authored-by: mystic-lama <mysticlama000@gmail.com>
Co-authored-by: Yiqun Zhang <guiyanakuang@gmail.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
---
 java/pom.xml | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/java/pom.xml b/java/pom.xml
index d83a113cb2..29e82bb018 100644
--- a/java/pom.xml
+++ b/java/pom.xml
@@ -66,9 +66,9 @@
     <java.version>17</java.version>
     <junit.version>5.10.0</junit.version>
     <maven-assembly-plugin.version>3.6.0</maven-assembly-plugin.version>
+    <maven-dependency-plugin.version>3.6.0</maven-dependency-plugin.version>
     <maven-shade-plugin.version>3.5.0</maven-shade-plugin.version>
-    <maven.compiler.source>1.8</maven.compiler.source>
-    <maven.compiler.target>1.8</maven.compiler.target>
+    <maven.compiler.release>17</maven.compiler.release>
     <maven.compiler.useIncrementalCompilation>false</maven.compiler.useIncrementalCompilation>
     <maven.version>3.9.4</maven.version>
 
@@ -445,7 +445,7 @@
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-dependency-plugin</artifactId>
-          <version>3.1.2</version>
+          <version>${maven-dependency-plugin.version}</version>
           <configuration>
             <failOnWarning>true</failOnWarning>
             <ignoreNonCompile>true</ignoreNonCompile>
@@ -801,14 +801,5 @@
         <module>bench</module>
       </modules>
     </profile>
-    <profile>
-      <id>java8</id>
-      <activation>
-        <jdk>1.8</jdk>
-      </activation>
-      <properties>
-        <checkstyle.version>9.3</checkstyle.version>
-      </properties>
-    </profile>
   </profiles>
 </project>