DomGarguilo edited a comment on issue #1760: URL: https://github.com/apache/accumulo/issues/1760#issuecomment-741905611
I was able to change the checkState version to include a more helpful message which now reads: `mytserveraddress ERROR The given address does not include a port`. > Based on this, it looks like most possibilities already have meaningful messages. So it may be only the missing port case that needs a better message. I don't think we need to keep the stack traces visible if that case is addressed. However, it may still be useful to log the exception at DEBUG anyway, when it is caught. @ctubbsii, by stack trace in this situation, do you mean the exceptions (e.g. `org.apache.thrift.transport.TTransportException: java.net.ConnectException: Connection refused`) included in the message, or were you referring to the full stack trace that I had included (and have since removed) in the initial revision of the print out when no port is passed? If its the prior, I can work to remove those from the error messages if that would be helpful and have the exceptions logged at DEBUG instead. ---------------------------------------------------------------- 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]
