Github user jbertram commented on the issue: https://github.com/apache/activemq-artemis/pull/2169 @michaelandrepearce, that constructor is used over 2,000 times in the code-base so deprecating it probably isn't the best option. I think we'd probably rather want to just move the null check from the toSimpleString method to the constructor itself.
---