Juan Hernandez has posted comments on this change. Change subject: core: Cache JNDI lookups and InitialContext creation ......................................................................
Patch Set 1: (1 inline comment) .................................................... File backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ejb/EJBUtilsStrategy.java Line 53: Line 54: if (context == null) Line 55: context = new InitialContext(); Line 56: Line 57: return context; Yes, makes sense for me. Line 58: } Line 59: Line 60: /** Line 61: * Finds a bean according to its JNDI name -- To view, visit http://gerrit.ovirt.org/16612 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I45ae8d0a90a03c498e19e5429d196924671c073b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Liran Zelkha <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
