leesf commented on a change in pull request #1114: [HUDI-438] Merge duplicated
code fragment
URL: https://github.com/apache/incubator-hudi/pull/1114#discussion_r363036834
##########
File path: hudi-spark/src/main/scala/org/apache/hudi/HoodieSparkSqlWriter.scala
##########
@@ -340,4 +247,72 @@ private[hudi] object HoodieSparkSqlWriter {
hiveSyncConfig.partitionValueExtractorClass =
parameters(HIVE_PARTITION_EXTRACTOR_CLASS_OPT_KEY)
hiveSyncConfig
}
+
+ /**
+ * Running into issues wrt generic type conversion from Java to Scala.
Couldn't make common code paths for
+ * write and deletes. Specifically, instantiating client of type
HoodieWriteClient<T extends HoodieRecordPayload>
+ * is having issues. Hence some codes blocks are same in both if and else
blocks.
Review comment:
Is this comment needed? could we remove 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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
