guoweiM commented on pull request #6613: URL: https://github.com/apache/flink/pull/6613#issuecomment-698073535
Sorry for the late reply. Thanks @lvhuyen very much for resolving the comments. What I concern is that why we need `globalModificationTime` and `maxProcessedTime` at the same time. Actually we could always compute the `globalModificationTime` from the `maxProcessedTime`. IMHO reducing the mutable states would make the code better maintenance and understanding. Of course maybe I miss some scenario which needs the two states. So would you like to enlighten me why we still need these two variables at the same time. Thanks again for your patience. ---------------------------------------------------------------- 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: [email protected]
