[ 
https://issues.apache.org/jira/browse/OPENJPA-407?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kevin Sutter updated OPENJPA-407:
---------------------------------

    Attachment: openjpa-407.patch

Attaching a new version of the first patch for this Issue.  All of the stated 
comments about the first patch have been addressed.  But, this is not the 
"final" patch.  There are still some areas that could use some improvement, but 
I wanted to get the basic updates in place so that we can move forward.  Also, 
by getting these into trunk, we can wring out any potential problems.

This patch introduces a property (modeled after the QueryCompilationCache 
property):

openjpa.jdbc.QuerySQLCache
o  true - uses CacheMap (default)
o  all - uses a ConcurrentHashMap
o  false - none

As before, this improvement attempts to cache the SelectImpls and associated 
SQLBuffers for common findBy operations.  The cache is scoped to an 
EntityManager instance.

The patch also includes new trace messages for cache hits and misses.

Any comments would be appreciated.  Like I mentioned, I would like to integrate 
this much of the patch soon since this is not a full-time Issue for me and it's 
a pain to continually update my environment.  If we're generally okay with the 
approach, then I'd like to commit and then move on to the next level of SQL 
generation caching.

Thanks,
Kevin

> Cache SQL (or closer precursors to SQL) more aggressively
> ---------------------------------------------------------
>
>                 Key: OPENJPA-407
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-407
>             Project: OpenJPA
>          Issue Type: Improvement
>          Components: jdbc, kernel, query, sql
>    Affects Versions: 0.9.0, 0.9.6, 0.9.7, 1.0.0
>            Reporter: Patrick Linskey
>             Fix For: 1.1.0
>
>         Attachments: findBy.patch, openjpa-407.patch, OPENJPA-407.patch
>
>
> When data is not available in the data cache, OpenJPA dynamically creates SQL 
> to look up the requested data. OpenJPA should more aggressively cache this 
> SQL to accelerate pathways from a cache miss to the database.
> The generated SQL takes a number of factors into account, including the 
> requested records, transaction status, currently-loaded data, and the current 
> fetch configuration. Any caching would need to account for these factors as 
> well.

-- 
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