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