ctubbsii commented on a change in pull request #1833:
URL: https://github.com/apache/accumulo/pull/1833#discussion_r539516116
##########
File path: core/src/main/java/org/apache/accumulo/core/util/HostAndPort.java
##########
@@ -105,7 +105,7 @@ public boolean hasPort() {
* occurring.
*/
public int getPort() {
- checkState(hasPort());
+ checkState(hasPort(), "The given address does not include a port");
Review comment:
Just a minor wording suggestion:
```suggestion
checkState(hasPort(), "the address does not include a port");
```
The reason for this is that the exception could also occur when the user has
not "given" any address. For example, if it's our own internal code. In that
case, the word "given" might be confusing.
----------------------------------------------------------------
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]
