linliu-code commented on code in PR #14317:
URL: https://github.com/apache/hudi/pull/14317#discussion_r2789264079
##########
hudi-aws/src/main/java/org/apache/hudi/config/GlueCatalogSyncClientConfig.java:
##########
@@ -89,4 +89,12 @@ public class GlueCatalogSyncClientConfig extends
HoodieConfig {
.withDocumentation(String.join(" ", "Specify the partitions fields to
index on aws glue. Separate the fields by semicolon.",
"By default, when the feature is enabled, all the partition will be
indexed.",
"You can create up to three indexes, separate them by comma. Eg:
col1;col2;col3,col2,col3"));
+
+ public static final ConfigProperty<Boolean> RECREATE_GLUE_TABLE_ON_ERROR =
ConfigProperty
+ .key(GLUE_CLIENT_PROPERTY_PREFIX + "recreate_table_on_error")
+ .defaultValue(false)
+ .sinceVersion("0.14.0")
Review Comment:
I think it should be 0.14.2 since we are going to land these change to
0.14.2.
##########
hudi-sync/hudi-hive-sync/src/main/java/org/apache/hudi/hive/HiveSyncConfig.java:
##########
@@ -88,6 +88,14 @@ public class HiveSyncConfig extends HoodieSyncConfig {
+ "filters exceed this size, will directly try to fetch all
partitions between the min/max."
+ "In case of glue metastore, this value should be reduced because
it has a filter length limit.");
+ public static final ConfigProperty<Boolean> RECREATE_HIVE_TABLE_ON_ERROR =
ConfigProperty
+ .key("hoodie.datasource.hive_sync.recreate_table_on_error")
+ .defaultValue(false)
+ .sinceVersion("0.14.0")
Review Comment:
I think it should be 0.14.2 since we are going to land these change to
0.14.2.
--
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]
