nsivabalan commented on code in PR #17531:
URL: https://github.com/apache/hudi/pull/17531#discussion_r2767908686


##########
.github/workflows/bot.yml:
##########
@@ -686,6 +686,17 @@ jobs:
           distribution: 'temurin'
           architecture: x64
           cache: maven
+      - name: Check disk space

Review Comment:
   can we move this into a separate PR?



##########
docker/compose/docker-compose_hadoop284_hive233_spark244.yml:
##########
@@ -175,7 +175,7 @@ services:
       - "namenode"
 
   zookeeper:
-    image: 'bitnami/zookeeper:3.4.12-r68'
+    image: 'bitnamilegacy/zookeeper:3.4.12-r68'

Review Comment:
   this one too



##########
hudi-spark-datasource/hudi-spark3.2plus-common/src/main/scala/org/apache/spark/sql/hudi/analysis/HoodieSpark32PlusAnalysis.scala:
##########
@@ -111,7 +111,13 @@ case class HoodieSpark32PlusResolveReferences(spark: 
SparkSession) extends Rule[
       lazy val analyzer = spark.sessionState.analyzer
       val targetTable = if (targetTableO.resolved) targetTableO else 
analyzer.execute(targetTableO)
       val sourceTable = if (sourceTableO.resolved) sourceTableO else 
analyzer.execute(sourceTableO)
-      val m = mO.asInstanceOf[MergeIntoTable].copy(targetTable = targetTable, 
sourceTable = sourceTable)
+      val originalMergeInto = mO.asInstanceOf[MergeIntoTable]
+      val m = originalMergeInto.copy(
+        targetTable = targetTable,
+        sourceTable = sourceTable,
+        mergeCondition = originalMergeInto.mergeCondition,
+        matchedActions = originalMergeInto.matchedActions,
+        notMatchedActions = originalMergeInto.notMatchedActions)

Review Comment:
   So, this is an issue only w/ spark 3.2 is it?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to