HuangZhenQiu commented on pull request #8952: URL: https://github.com/apache/flink/pull/8952#issuecomment-752331980
@xintongsong In the current implementation, I choose to use the class name TimestampBasedFailureRater rather than ThresholdMeter because there is a container failure related error message and config key are used in the class. I think it is not generic enough to name it as ThresholdMeter. For the formatting rules. I used spotless to format the code after rebasing the master. The code should already conform to the rule proposed. ---------------------------------------------------------------- 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]
