[GitHub] [qpid-jms] gemmellr commented on pull request #40: Remove InetAddress.getByName lookup for compare

[GitBox <gi...@apache.org>]

gemmellr commented on pull request #40:
URL: https://github.com/apache/qpid-jms/pull/40#issuecomment-825572636


   Please raise a JIRA (https://issues.apache.org/jira/browse/QPIDJMS) and update your commit (squash them) and the PR title to reference it  
   
   The Travis build was cancelled by a subsequent run starting, though normally that would then be reported instead, so its not clear how that happened.
   
   I believe this behaviour was deliberate and is certainly very long standing (inspired by an even older codebase), from times before cases like described really existed, so we will need to consider if we simply change it wholesale or need to make it configurable. The former would certainly be simpler.


-- 
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org