Github user brunobat commented on a diff in the pull request:

    https://github.com/apache/tomee/pull/332#discussion_r244574001
  
    --- Diff: 
container/openejb-core/src/main/java/org/apache/openejb/OpenEjbContainer.java 
---
    @@ -221,12 +218,13 @@ private static Logger logger() { // don't trigger 
init too eagerly to be sure to
         }
     
         public static class Provider implements EJBContainerProvider {
    +
             public static final String OPENEJB_ADDITIONNAL_CALLERS_KEY = 
"openejb.additionnal.callers";
     
             @Override
             public EJBContainer createEJBContainer(Map<?, ?> map) {
    -            if (map == null) { // JBoss EJB API pass null when calling 
EJBContainer.createEJBContainer()
    -                map = new HashMap<>();
    +            if (map == null) {
    +                map = new HashMap<Object, Object>();
    --- End diff --
    
    We should use the diamond operator only. 


---

Reply via email to