Skip to content

Commit

Permalink
Remove check for Java versions
Browse files Browse the repository at this point in the history
  • Loading branch information
Chong Gao committed Dec 25, 2024
1 parent 36d3cdf commit 5bfd763
Showing 1 changed file with 0 additions and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -140,14 +140,12 @@ object HybridFileSourceScanExecMeta {
* Check if runtimes are satisfied, including:
* - Spark distribution is not CDH or Databricks
* - Hybrid jar in the classpath
* - Java version is 1.8
* - Scala version is 2.12
* - Parquet V1 data source
*/
def checkRuntimes(v1DataSourceList: String): Unit = {
checkNotRunningCDHorDatabricks()
HybridExecutionUtils.checkHybridJarInClassPath()
checkJavaVersion()
checkScalaVersion()
checkV1Datasource(v1DataSourceList)
}
Expand All @@ -164,23 +162,6 @@ object HybridFileSourceScanExecMeta {
}
}

/**
* Hybrid feature only supports 1.8 Java version,
* report error if not
*/
private def checkJavaVersion(): Unit = {
val javaVersion = System.getProperty("java.version")
if (javaVersion == null) {
throw new RuntimeException("Hybrid feature: Can not read java.version, get null")
}
if (!javaVersion.startsWith("1.8")) {
throw new RuntimeException(
"Hybrid jar is not in the classpath, Please add Hybrid jar into the class path, or " +
"Please disable Hybrid feature by setting " +
"spark.rapids.sql.parquet.useHybridReader=false")
}
}

/**
* Hybrid feature only supports Scala 2.12 version,
* report error if not
Expand Down

0 comments on commit 5bfd763

Please sign in to comment.