zhuanshenbsj1 commented on code in PR #12278:
URL: https://github.com/apache/hudi/pull/12278#discussion_r1846505922


##########
hudi-flink-datasource/hudi-flink/src/main/java/org/apache/hudi/source/StreamReadMonitoringFunction.java:
##########
@@ -234,6 +235,9 @@ public void 
monitorDirAndForwardSplits(SourceContext<MergeOnReadInputSplit> cont
             + "---------- consumed to instant: {}\n"
             + "------------------------------------------------------------",
         conf.getString(FlinkOptions.TABLE_NAME), this.issuedInstant);
+    if (result.isEmpty()) {
+      LOG.warn("In the current round-robin stream read, there are no files to 
be read in these commits.");

Review Comment:
   > And this log is redundant with line 221, we should just keep one of them.
   
   Line 221 means that there are no commits to read for current run. 
   Here means there has somes commits to read,  but there are no readable data 
files for these commits(maybe empty commits or alter table commits),  and a 
line of prompt information is given to the user.



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