leesf commented on a change in pull request #2324:
URL: https://github.com/apache/hudi/pull/2324#discussion_r541826761
##########
File path:
hudi-sync/hudi-dla-sync/src/main/java/org/apache/hudi/dla/DLASyncConfig.java
##########
@@ -62,6 +62,9 @@
@Parameter(names = {"--skip-ro-suffix"}, description = "Skip the `_ro`
suffix for Read optimized table, when registering")
public Boolean skipROSuffix = false;
+ @Parameter(names = {"--skip-rt-sync"}, description = "Skip the RT table
syncing")
+ public Boolean skipRTSync = true;
Review comment:
just one more comment, here the default value to be false would be
better, since some users have synced both ro and rt tables to dla.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
