[ 
https://issues.apache.org/jira/browse/HUDI-1105?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17375833#comment-17375833
 ] 

ASF GitHub Bot commented on HUDI-1105:
--------------------------------------

nsivabalan commented on a change in pull request #2206:
URL: https://github.com/apache/hudi/pull/2206#discussion_r664705394



##########
File path: 
hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/hudi/HoodieSparkSqlWriter.scala
##########
@@ -333,7 +334,9 @@ private[hudi] object HoodieSparkSqlWriter {
     log.info(s"Registered avro schema : ${schema.toString(true)}")
     val params = parameters.updated(HoodieWriteConfig.AVRO_SCHEMA, 
schema.toString)
     val writeConfig = DataSourceUtils.createHoodieConfig(schema.toString, 
path.get, tblName, mapAsJavaMap(params))
-    val hoodieDF = 
HoodieDatasetBulkInsertHelper.prepareHoodieDatasetForBulkInsert(sqlContext, 
writeConfig, df, structName, nameSpace)
+    val isGlobalIndex = SparkHoodieIndex.createIndex(writeConfig).isGlobal

Review comment:
       may be for out of the box index types, we can maintain a static map of 
IndexType to isGlobal and use it. wdyt? 




-- 
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]


> Bulk insert dataset - Dedup
> ---------------------------
>
>                 Key: HUDI-1105
>                 URL: https://issues.apache.org/jira/browse/HUDI-1105
>             Project: Apache Hudi
>          Issue Type: Sub-task
>          Components: Writer Core
>    Affects Versions: 0.9.0
>            Reporter: sivabalan narayanan
>            Assignee: sivabalan narayanan
>            Priority: Blocker
>              Labels: pull-request-available
>             Fix For: 0.9.0
>
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to