DomGarguilo commented on code in PR #5966:
URL: https://github.com/apache/accumulo/pull/5966#discussion_r2561232655
##########
core/src/main/java/org/apache/accumulo/core/conf/PropertyType.java:
##########
@@ -423,43 +420,71 @@ public boolean test(final String input) {
}
- public static class PortRange extends Matches {
-
- public static final Range<Integer> VALID_RANGE = Range.of(1024, 65535);
-
- public PortRange(final String pattern) {
- super(pattern);
- }
+ private static class PortPredicate implements Predicate<String> {
@Override
public boolean test(final String input) {
- if (super.test(input)) {
+ if (input == null) {
+ return true;
+ }
+ final String trimmed = input.trim();
+ if (trimmed.isEmpty()) {
+ return false;
+ }
+ if ("0".equals(trimmed)) {
+ return true;
+ }
Review Comment:
Added to the docs for RPC_BIND_PORT in
d6d85530ffae62d969127195612e1a1d7b6890a7
--
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]
