[
https://issues.apache.org/jira/browse/OPENJPA-1741?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12897470#action_12897470
]
David Minor commented on OPENJPA-1741:
--------------------------------------
Yes. In periodic thread dumps of our production JVM, the number of OpenJPA
threads in getPathOrConstant() went from 25% of OpenJPA threads to 3%.
> Unecessary class load attempts when parsing JPQL
> ------------------------------------------------
>
> Key: OPENJPA-1741
> URL: https://issues.apache.org/jira/browse/OPENJPA-1741
> Project: OpenJPA
> Issue Type: Bug
> Components: query
> Affects Versions: 1.2.2, 2.0.0
> Reporter: David Minor
> Attachments: cache-class-load-attempts.patch,
> only-load-uppercase-java-lang.patch, skip-known-variables.patch
>
>
> When parsing JPQL, JPQLExpressionBuilder.getPathOrConstant() tries to
> determine if a token is a java class or enum by doing a class load (via
> QueryImpl.classForName()). The same token will be tried as often as it
> appears in the query. It might be better to
> 1) cache the results
> 2) avoid the lookup if the token is a known variable name
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
