Github user gemmellr commented on the pull request:

    https://github.com/apache/activemq-artemis/pull/88#issuecomment-123631457
  
    I'd suggest using the ConcurrentMap interface for the declarations rather 
than using the ConcurrentHashMap type directly, otherwise you can introduce 
scenarios where binaries produced from source compiled with JDK 8 targetting 
JDK7 won't actually work on JDK7 even though it is legal source and would work 
if compiled using JDK7.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to