curcur commented on a change in pull request #14881:
URL: https://github.com/apache/flink/pull/14881#discussion_r571443770



##########
File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/state/AbstractKeyedStateBackend.java
##########
@@ -127,7 +127,9 @@ public AbstractKeyedStateBackend(
                 numberOfKeyGroups >= 1, "NumberOfKeyGroups must be a positive 
number");
         Preconditions.checkArgument(
                 numberOfKeyGroups >= keyGroupRange.getNumberOfKeyGroups(),
-                "The total number of key groups must be at least the number in 
the key group range assigned to this backend");
+                "The total number of key groups must be at least the number in 
the key group range assigned to this backend %s %s",

Review comment:
       Hmm, maybe even more specific: 
   "The total number of key groups must be at least the number in the key group 
range assigned to this backend. The total number of key groups=%s ; the number 
in the key group range=%s"
   
   Or something similar?




----------------------------------------------------------------
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]


Reply via email to