[ https://issues.apache.org/jira/browse/MYFACES-4330?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17085977#comment-17085977 ]
Thomas Andraschko commented on MYFACES-4330: -------------------------------------------- Great patch! DO you have some numbers of the performance improvements? Nevertheless, you can create PRs here: [https://github.com/apache/myfaces] Please note: we need at least a PR for master, 2.3.x and 3.0.x If you would like to see it in 2.2, we also need a PR for the 2.2.x branch > Bad performance of ResourceHandlerImpl because of unnecessary > MissingResourceException > -------------------------------------------------------------------------------------- > > Key: MYFACES-4330 > URL: https://issues.apache.org/jira/browse/MYFACES-4330 > Project: MyFaces Core > Issue Type: Bug > Affects Versions: 2.2.12 > Reporter: Reto Weiss > Priority: Major > Attachments: myfaces.patch > > > While processing JSF resource requests a lot of MissingResourceExceptions are > thrown and catched right afterwards if you have a message bundle installed > but no resource for the key 'javax.faces.resource.localePrefix' is provided > (like in the case of Primefaces). > This can lead to performance problems because creating and throwing an > exception is a heavy operation. The throwing of the exception can be > prevented by first checking if a resource contains a certain resource before > accessing it: > try { > ResourceBundle bundle = ResourceBundle > .getBundle(bundleName, locale, > ClassUtils.getContextClassLoader()); if (bundle != null) > { if > (bundle.containsKey(ResourceHandler.LOCALE_PREFIX)) > \{ > localePrefix = > bundle.getString(ResourceHandler.LOCALE_PREFIX); > } > } > } catch (MissingResourceException e) > { // Ignore it and return null } > By applying the above patch I see a 50% reduction of the execution time of > the method createResource. -- This message was sent by Atlassian Jira (v8.3.4#803005)