zhijiangW commented on pull request #2:
URL: https://github.com/apache/flink-benchmarks/pull/2#issuecomment-678889692


   > But the problem with the current setup is that if checkpointing time is 
small, you will be measuring just the time to complete last checkpoint, instead 
of 5. And the problem is that even if the current interval makes sense, the 
checkpointing time can change in the future, as we improve things, which would 
render this benchmark obsolete.
   
   You are right. I misunderstood the semantic of checkpoint interval before. I 
wrongly thought that the internal was only for triggering the first checkpoint, 
and the following checkpoint will be triggered immediately after the preceding 
one completes. As you said, internal `0` should be proper for voiding delays 
for intermediate checkpoints. 


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to