nsivabalan commented on code in PR #13214:
URL: https://github.com/apache/hudi/pull/13214#discussion_r2056252489
##########
hudi-common/src/main/java/org/apache/hudi/common/config/HoodieMetadataConfig.java:
##########
@@ -381,9 +381,10 @@ public final class HoodieMetadataConfig extends
HoodieConfig {
public static final ConfigProperty<Boolean>
ENABLE_METADATA_INDEX_PARTITION_STATS = ConfigProperty
.key(METADATA_PREFIX + ".index.partition.stats.enable")
- .defaultValue(false)
+ .defaultValue(true)
Review Comment:
we do this at per engine level. thats why we did not flip this value.
if you check HoodieMetadataConfig,
we have getDefaultColStatsEnable() which will choose the right value for col
stats for different engines.
and the default value for partitions stats enable goes along w/ the derived
value for col stats.
if col stats is enabled, PSI will also be enabled. if not, it will not.
We just need to update our release docs to call this out.
--
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]
