YOMO-Lee commented on code in PR #7921:
URL: https://github.com/apache/seatunnel/pull/7921#discussion_r1817959446


##########
seatunnel-engine/seatunnel-engine-common/src/main/resources/seatunnel.yaml:
##########
@@ -20,6 +20,7 @@ seatunnel:
         backup-count: 1
         queue-type: blockingqueue
         print-execution-info-interval: 60
+        print-job-metrics-info-interval: 60

Review Comment:
   > Default value of `print-job-metrics-info-interval` is 60. This is valid 
without change
   
   I think this parameter can be added here, at least adding the default value 
will not cause any other bad effects
   But it worked for me, at first I wanted to change the configuration, I 
didn't know where to change, I searched a lot of places in the project, and it 
didn't take effect after the change, and then it was someone else who pointed 
out that I knew to change here



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