[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-12 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1164679885 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieBootstrapConfig.java: ## @@ -83,8 +79,8 @@ public class HoodieBootstrapConfig extends HoodieCon

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163391536 ## hudi-common/src/test/java/org/apache/hudi/common/config/TestConfigProperty.java: ## @@ -171,4 +171,28 @@ public void testAdvancedValue() { assertTrue(FAKE_BOOLEAN_C

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163375572 ## hudi-common/src/main/java/org/apache/hudi/common/config/ConfigProperty.java: ## @@ -139,6 +144,49 @@ public ConfigProperty withDocumentation(String doc) { return ne

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163352137 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ## @@ -168,23 +168,16 @@ public class HoodieWriteConfig extends HoodieConfig

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163346107 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieClusteringConfig.java: ## @@ -732,40 +712,17 @@ public String getValue() { } } + @En

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163302008 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/bootstrap/BootstrapMode.java: ## @@ -18,18 +18,28 @@ package org.apache.hudi.client.bootstrap; +

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-11 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1163153414 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ## @@ -194,16 +187,18 @@ public class HoodieWriteConfig extends HoodieConfig

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-03 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1156300665 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ## @@ -2929,12 +2892,12 @@ private void autoAdjustConfigsForConcurrencyMode(

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-03 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1156299374 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieCleanConfig.java: ## @@ -64,25 +62,20 @@ public class HoodieCleanConfig extends HoodieConfig {

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-04-03 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1156298274 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieBootstrapConfig.java: ## @@ -55,12 +52,10 @@ public class HoodieBootstrapConfig extends HoodieC

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106023838 ## hudi-common/src/main/java/org/apache/hudi/keygen/constant/KeyGeneratorType.java: ## @@ -18,47 +18,41 @@ package org.apache.hudi.keygen.constant; +import org.apache.

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106023367 ## hudi-common/src/main/java/org/apache/hudi/common/util/queue/ExecutorType.java: ## @@ -18,27 +18,27 @@ package org.apache.hudi.common.util.queue; +import org.apache.

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106022582 ## hudi-common/src/main/java/org/apache/hudi/common/util/queue/DisruptorWaitStrategyType.java: ## @@ -27,35 +30,50 @@ /** * Enum for the type of waiting strategy in Disr

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106021736 ## hudi-common/src/main/java/org/apache/hudi/common/table/marker/MarkerType.java: ## @@ -19,10 +19,23 @@ package org.apache.hudi.common.table.marker; +import org.apach

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106016160 ## hudi-common/src/main/java/org/apache/hudi/common/bloom/BloomFilterTypeCode.java: ## @@ -18,11 +18,21 @@ package org.apache.hudi.common.bloom; +import org.apache.hud

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106015149 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/storage/HoodieStorageLayout.java: ## @@ -50,7 +53,14 @@ public HoodieStorageLayout(HoodieWriteConfig c

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106014556 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/compact/CompactionTriggerStrategy.java: ## @@ -18,15 +18,31 @@ package org.apache.hudi.table

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106012580 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/cluster/ClusteringPlanPartitionFilterMode.java: ## @@ -18,12 +18,31 @@ package org.apache.hu

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106011282 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/clean/CleaningTriggerStrategy.java: ## @@ -18,7 +18,14 @@ package org.apache.hudi.table.acti

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106009875 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/index/HoodieIndex.java: ## @@ -152,11 +155,51 @@ public boolean requiresTagging(WriteOperationType operatio

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106008290 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ## @@ -1912,6 +1871,10 @@ public Option getLogDataBlockFormat() { .

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106007724 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieWriteConfig.java: ## @@ -187,16 +178,18 @@ public class HoodieWriteConfig extends HoodieConfig

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106006641 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieIndexConfig.java: ## @@ -74,11 +74,10 @@ public class HoodieIndexConfig extends HoodieConfig {

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106005114 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieClusteringConfig.java: ## @@ -692,9 +687,17 @@ public static SpatialCurveCompositionStrategyTyp

[GitHub] [hudi] jonvex commented on a diff in pull request #7881: [HUDI-5723] Automate and standardize enum configs

2023-02-14 Thread via GitHub
jonvex commented on code in PR #7881: URL: https://github.com/apache/hudi/pull/7881#discussion_r1106001981 ## hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/bootstrap/BootstrapMode.java: ## @@ -18,18 +18,27 @@ package org.apache.hudi.client.bootstrap; +