prashantwason commented on code in PR #17550:
URL: https://github.com/apache/hudi/pull/17550#discussion_r2662663821


##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/config/HoodieCleanConfig.java:
##########
@@ -201,6 +201,11 @@ public class HoodieCleanConfig extends HoodieConfig {
           + "table receives updates/deletes. Another reason to turn this on, 
would be to ensure data residing in bootstrap "
           + "base files are also physically deleted, to comply with data 
privacy enforcement processes.");
 
+  public static final ConfigProperty<String> CLEANER_PARTITION_FILTER_REGEX = 
ConfigProperty
+      .key("hoodie.cleaner.partition.filter.regex")

Review Comment:
   Separated the config into two as per the handling on other parts of HUDI 
code. The same config taking in regex and static values would be confusing and 
may cause issues.



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

Reply via email to