xiarixiaoyao commented on a change in pull request #3209:
URL: https://github.com/apache/hudi/pull/3209#discussion_r663445355
##########
File path:
hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/DataSourceOptions.scala
##########
@@ -462,6 +462,11 @@ object DataSourceWriteOptions {
.defaultValue(false)
.withDocumentation("Whether to sync the table as managed table.")
+ val HIVE_BATCH_SYNC_PARTITION_NUM: ConfigProperty[String] = ConfigProperty
Review comment:
oh, i see many boolean type config property is set to be
ConfigProperty[String], so i follow this style, make this config to String
in scala, it's better to set it Int instead of Integer. fix the comment.
--
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]
