[ 
https://issues.apache.org/jira/browse/OPENJPA-160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12475455
 ] 

Michael Dick commented on OPENJPA-160:
--------------------------------------

We ran some more performance tests with the latest OpenJPA code and the issue 
appears to be with creating an instance of the BrokerImpl (when Configurations 
calls Class.newInstance). 

I was surprised that creating a new instance turned out to take so much time 
and  I don't know what we'd could (or would want to) tinker with to try to make 
it faster to create. I'm not thrilled about adding the complexity of a reuse 
pool so I'm open to suggestions. 

The pool that we used before was a two level pool thread.toString+user+pass -> 
collection of brokers. Adding a non static field to AbstractBrokerFactory 
sounds feasible too (unless there's an alternative to pooling).

Still looking into whether we need a key in BrokerImpl - I'll follow up on that 
as well. 

> Reuse BrokerImpl objects
> ------------------------
>
>                 Key: OPENJPA-160
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-160
>             Project: OpenJPA
>          Issue Type: Sub-task
>            Reporter: Michael Dick
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to