Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5676#discussion_r174446646
--- Diff:
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/operators/BackendRestorerProcedure.java
---
@@ -105,6 +105,8 @@ public T createAndRestore(@Nonnull List<? extends
Collection<S>> restoreOptions)
++alternativeIdx;
+ // IMPORTANT: please be careful when modifying the log
statements because they are used for validation in
+ // the automatic end-to-end tests. Those tests might
fail if they are not aligned with the log message!
if (restoreState.isEmpty()) {
LOG.debug("Creating {} with empty state.",
logDescription);
} else {
--- End diff --
I feel that guarding a log statement format by a test, just so that another
test won't fail is a bit too much. I mean, it does not really break anything so
a comment should be ok.
---
