You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by "danny0405 (via GitHub)" <gi...@apache.org> on 2023/02/09 09:23:50 UTC

[GitHub] [hudi] danny0405 commented on a diff in pull request #7903: [HUDI-5734]Fix flink batch read skip clustering data lost

danny0405 commented on code in PR #7903:
URL: https://github.com/apache/hudi/pull/7903#discussion_r1101181519


##########
hudi-flink-datasource/hudi-flink/src/test/java/org/apache/hudi/source/TestIncrementalInputSplits.java:
##########
@@ -57,6 +64,7 @@ void testFilterInstantsWithRange() {
         .conf(conf)
         .path(new Path(basePath))
         .rowType(TestConfigurations.ROW_TYPE)
+        .skipClustering(true)

Review Comment:
   We can config the `skip_xxx` with the option value from the configuration, just like what we do for streaming read.



-- 
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: commits-unsubscribe@hudi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org