[jboss-dev-forums] [Design the new POJO MicroContainer] - KernelRegistry - unnecessary Exception construction

[adrian@jboss.org]

I'm seeing this as a "hotspot" during bootstrap.


  |         org.jboss.kernel.plugins.registry.AbstractKernelRegistry.getEntry(AbstractKernelRegistry.java:89)
  |         org.jboss.kernel.plugins.dependency.AbstractKernelController.getContext(AbstractKernelController.java:154)
  |         org.jboss.dependency.plugins.AbstractDependencyItem.resolve(AbstractDependencyItem.java:128)
  |         org.jboss.dependency.plugins.AbstractDependencyInfo.resolveDependencies(AbstractDependencyInfo.java:143)
  |         org.jboss.dependency.plugins.AbstractController.resolveContexts(AbstractController.java:951)
  |         org.jboss.dependency.plugins.AbstractController.resolveContexts(AbstractController.java:889)
  |         org.jboss.dependency.plugins.AbstractController.resolveContexts(AbstractController.java:834)
  |         org.jboss.dependency.plugins.AbstractController.change(AbstractController.java:672)
  |         org.jboss.dependency.plugins.AbstractController.change(AbstractController.java:455)
  |         org.jboss.system.ServiceController.doChange(ServiceController.java:664)
  | 

The reason is that it gets called a lot and throws
KernelRegistryEntryNotFoundException
when the context is not installed yet.

Absent doing the work to remove the deprecated KernelReigstry
we should probably just add a findEntry() method
that returns null and use that from the KernelController.

The only reason this call exists is because of the KernelRegistryPlugins
support for which should be moved to the KernelController.

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4166363#4166363

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4166363