sollhui commented on code in PR #56930:
URL: https://github.com/apache/doris/pull/56930#discussion_r2428098137


##########
fe/fe-core/src/main/java/org/apache/doris/load/routineload/RoutineLoadTaskInfo.java:
##########
@@ -186,6 +188,21 @@ private void judgeEof(RLTaskTxnCommitAttachment 
rlTaskTxnCommitAttachment) {
         }
     }
 
+    private void adaptiveMaxBatchInterval(RLTaskTxnCommitAttachment 
rlTaskTxnCommitAttachment) {
+        RoutineLoadJob routineLoadJob = routineLoadManager.getJob(jobId);
+        if (rlTaskTxnCommitAttachment.getTotalRows() < 
routineLoadJob.getMaxBatchRows()
+                && rlTaskTxnCommitAttachment.getReceivedBytes() < 
routineLoadJob.getMaxBatchSizeBytes()
+                && rlTaskTxnCommitAttachment.getTaskExecutionTimeMs() >= 
routineLoadJob.getMaxBatchIntervalS()) {

Review Comment:
   Not exactly the same. For example, if reach the finish condition by 
`max_bathc_size/rows`, we don't think it is EOF and also not increase 
`max_batch_interval` adaptively.



-- 
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-unsubscr...@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to