davidradl commented on code in PR #26460:
URL: https://github.com/apache/flink/pull/26460#discussion_r2043946263
##########
flink-core/src/main/java/org/apache/flink/util/NetUtils.java:
##########
@@ -423,7 +423,7 @@ public static Iterator<Integer>
getPortRangeFromString(String rangeDefinition)
final int port = Integer.parseInt(range);
if (!isValidHostPort(port)) {
throw new IllegalConfigurationException(
- "Invalid port configuration. Port must be between
0"
+ "Invalid port configuration. Port must be between
0 "
Review Comment:
This is a trivial change, but if we are making it , I suggest making the
following change to the words as well:
65535 is the maximum so between is not inclusive, I would say` in the range`
as per the isValidHostPort javadoc.
Same for the other messages.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
