openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Sutter (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (OPENJPA-2414) FinderCache does not consider active Fetch Groups/FetchPlan added Fields
Date Wed, 28 Aug 2013 15:16:52 GMT

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

Kevin Sutter commented on OPENJPA-2414:
---------------------------------------

Oh sure, Thanks Jody for hauling me into this discussion...  :-)

I will agree that the current usage scenario was not intuitive or documented.  And, since
we have customers hitting this condition of accidentally caching the "wrong" SQL, I agree
that we need to do something in the service streams to resolve the issue.  And, considering
the 80/20 rule, it seems that erring on the default/conservative side definitely hits the
80% side of the market.

That all being said, Pinaki does have a valid point that we are now alienating users that
use fetch plans from using the FinderCache.  But, that is also consistent with Pinaki's direction
to have the application control whether these generated SQL's should be cached or not.  We're
just making that decision for them.

Also, does this change in behavior have any impact on the normal callpath from a performance
perspective?  Pinaki is hinting that it will affect performance.  If we are doing additional
processing on every generated SQL and access to the FinderCache, then are we negating the
benefits of the cache?

One alternative is to modify the FinderCache so that it could take the FetchPlan settings
into account.  Whether this extra processing would offset the benefit of the cache would have
to be determined.  Regardless, this is too big of an effort for the service streams.  I would
suggest creating a sub-JIRA feature for this effort so that we have it on the books.  But,
stick with this conservative approach until this sub-feature is resolved.

That's my two cents worth.  But, don't just re-close this JIRA without coming to some type
of agreement.  I've posted a few questions that should be discussed and resolved.  Thanks.

                
> FinderCache does not consider active Fetch Groups/FetchPlan added Fields
> ------------------------------------------------------------------------
>
>                 Key: OPENJPA-2414
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-2414
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: kernel
>    Affects Versions: 2.3.0, 2.2.3
>            Reporter: Jody Grassel
>            Assignee: Jody Grassel
>             Fix For: 2.1.2, 2.3.0, 2.2.1.1, 2.2.3
>
>
> The FinderCache retains a Map, associating a ClassMapping with a FinderQuery.  However,
this cache does not factor in the characteristics of the FetchPlan that was active when a
mapping is created, nor does it factor them to determine if a cache hit is appropriate.  This
causes the find() operation to perform the same SQL as the first time it was executed, regardless
of changes to the active FetchPlan afterwards.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message