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

ulyssesyou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 6dc7885f6 [CORE] Move BackendBuildInfo case class from GlutenPlugin to 
Backend class file (#5129)
6dc7885f6 is described below

commit 6dc7885f6c54f4ea0f773e920fb455e09298b3b7
Author: Zhen Wang <643348...@qq.com>
AuthorDate: Wed Mar 27 09:15:13 2024 +0800

    [CORE] Move BackendBuildInfo case class from GlutenPlugin to Backend class 
file (#5129)
---
 .../io/glutenproject/backendsapi/clickhouse/CHBackend.scala    |  6 +++---
 .../io/glutenproject/backendsapi/velox/VeloxBackend.scala      |  6 +++---
 gluten-core/src/main/scala/io/glutenproject/GlutenPlugin.scala |  6 ------
 .../src/main/scala/io/glutenproject/backendsapi/Backend.scala  | 10 +++++++---
 .../io/glutenproject/backendsapi/BackendsApiManager.scala      |  4 +---
 5 files changed, 14 insertions(+), 18 deletions(-)

diff --git 
a/backends-clickhouse/src/main/scala/io/glutenproject/backendsapi/clickhouse/CHBackend.scala
 
b/backends-clickhouse/src/main/scala/io/glutenproject/backendsapi/clickhouse/CHBackend.scala
index fbcb804a3..a7c5c9980 100644
--- 
a/backends-clickhouse/src/main/scala/io/glutenproject/backendsapi/clickhouse/CHBackend.scala
+++ 
b/backends-clickhouse/src/main/scala/io/glutenproject/backendsapi/clickhouse/CHBackend.scala
@@ -16,7 +16,7 @@
  */
 package io.glutenproject.backendsapi.clickhouse
 
-import io.glutenproject.{CH_BRANCH, CH_COMMIT, GlutenConfig, GlutenPlugin}
+import io.glutenproject.{CH_BRANCH, CH_COMMIT, GlutenConfig}
 import io.glutenproject.backendsapi._
 import io.glutenproject.expression.WindowFunctionsBuilder
 import io.glutenproject.extension.ValidationResult
@@ -41,8 +41,8 @@ import scala.util.control.Breaks.{break, breakable}
 
 class CHBackend extends Backend {
   override def name(): String = CHBackend.BACKEND_NAME
-  override def buildInfo(): GlutenPlugin.BackendBuildInfo =
-    GlutenPlugin.BackendBuildInfo("ClickHouse", CH_BRANCH, CH_COMMIT, 
"UNKNOWN")
+  override def buildInfo(): BackendBuildInfo =
+    BackendBuildInfo("ClickHouse", CH_BRANCH, CH_COMMIT, "UNKNOWN")
   override def iteratorApi(): IteratorApi = new CHIteratorApi
   override def sparkPlanExecApi(): SparkPlanExecApi = new CHSparkPlanExecApi
   override def transformerApi(): TransformerApi = new CHTransformerApi
diff --git 
a/backends-velox/src/main/scala/io/glutenproject/backendsapi/velox/VeloxBackend.scala
 
b/backends-velox/src/main/scala/io/glutenproject/backendsapi/velox/VeloxBackend.scala
index 0ff2bd0d7..3293abe3e 100644
--- 
a/backends-velox/src/main/scala/io/glutenproject/backendsapi/velox/VeloxBackend.scala
+++ 
b/backends-velox/src/main/scala/io/glutenproject/backendsapi/velox/VeloxBackend.scala
@@ -16,7 +16,7 @@
  */
 package io.glutenproject.backendsapi.velox
 
-import io.glutenproject.{GlutenConfig, GlutenPlugin, VELOX_BRANCH, 
VELOX_REVISION, VELOX_REVISION_TIME}
+import io.glutenproject.{GlutenConfig, VELOX_BRANCH, VELOX_REVISION, 
VELOX_REVISION_TIME}
 import io.glutenproject.backendsapi._
 import io.glutenproject.exception.GlutenNotSupportException
 import io.glutenproject.execution.WriteFilesExecTransformer
@@ -44,8 +44,8 @@ import scala.util.control.Breaks.breakable
 
 class VeloxBackend extends Backend {
   override def name(): String = VeloxBackend.BACKEND_NAME
-  override def buildInfo(): GlutenPlugin.BackendBuildInfo =
-    GlutenPlugin.BackendBuildInfo("Velox", VELOX_BRANCH, VELOX_REVISION, 
VELOX_REVISION_TIME)
+  override def buildInfo(): BackendBuildInfo =
+    BackendBuildInfo("Velox", VELOX_BRANCH, VELOX_REVISION, 
VELOX_REVISION_TIME)
   override def iteratorApi(): IteratorApi = new IteratorApiImpl
   override def sparkPlanExecApi(): SparkPlanExecApi = new SparkPlanExecApiImpl
   override def transformerApi(): TransformerApi = new TransformerApiImpl
diff --git a/gluten-core/src/main/scala/io/glutenproject/GlutenPlugin.scala 
b/gluten-core/src/main/scala/io/glutenproject/GlutenPlugin.scala
index c54b78da9..5fa3083c2 100644
--- a/gluten-core/src/main/scala/io/glutenproject/GlutenPlugin.scala
+++ b/gluten-core/src/main/scala/io/glutenproject/GlutenPlugin.scala
@@ -278,10 +278,4 @@ private[glutenproject] object GlutenPlugin {
   implicit def sparkConfImplicit(conf: SparkConf): SparkConfImplicits = {
     new SparkConfImplicits(conf)
   }
-
-  case class BackendBuildInfo(
-      backend: String,
-      backendBranch: String,
-      backendRevision: String,
-      backendRevisionTime: String)
 }
diff --git 
a/gluten-core/src/main/scala/io/glutenproject/backendsapi/Backend.scala 
b/gluten-core/src/main/scala/io/glutenproject/backendsapi/Backend.scala
index 438194a36..09799cdb1 100644
--- a/gluten-core/src/main/scala/io/glutenproject/backendsapi/Backend.scala
+++ b/gluten-core/src/main/scala/io/glutenproject/backendsapi/Backend.scala
@@ -16,12 +16,10 @@
  */
 package io.glutenproject.backendsapi
 
-import io.glutenproject.GlutenPlugin
-
 trait Backend {
   def name(): String
 
-  def buildInfo(): GlutenPlugin.BackendBuildInfo
+  def buildInfo(): BackendBuildInfo
 
   def iteratorApi(): IteratorApi
 
@@ -39,3 +37,9 @@ trait Backend {
 
   def settings(): BackendSettingsApi
 }
+
+case class BackendBuildInfo(
+    backend: String,
+    backendBranch: String,
+    backendRevision: String,
+    backendRevisionTime: String)
diff --git 
a/gluten-core/src/main/scala/io/glutenproject/backendsapi/BackendsApiManager.scala
 
b/gluten-core/src/main/scala/io/glutenproject/backendsapi/BackendsApiManager.scala
index bd896c3dd..867343bd3 100644
--- 
a/gluten-core/src/main/scala/io/glutenproject/backendsapi/BackendsApiManager.scala
+++ 
b/gluten-core/src/main/scala/io/glutenproject/backendsapi/BackendsApiManager.scala
@@ -16,8 +16,6 @@
  */
 package io.glutenproject.backendsapi
 
-import io.glutenproject.GlutenPlugin
-
 import java.util.ServiceLoader
 
 import scala.collection.JavaConverters
@@ -57,7 +55,7 @@ object BackendsApiManager {
     backend.name()
   }
 
-  def getBuildInfo: GlutenPlugin.BackendBuildInfo = {
+  def getBuildInfo: BackendBuildInfo = {
     backend.buildInfo()
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@gluten.apache.org
For additional commands, e-mail: commits-h...@gluten.apache.org

Reply via email to