You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by "Jeremy Bauer (JIRA)" <ji...@apache.org> on 2008/10/16 18:19:44 UTC

[jira] Created: (OPENJPA-748) Disable QuerySQLCache by default

Disable QuerySQLCache by default
--------------------------------

                 Key: OPENJPA-748
                 URL: https://issues.apache.org/jira/browse/OPENJPA-748
             Project: OpenJPA
          Issue Type: Bug
          Components: jdbc
    Affects Versions: 1.2.0, 1.3.0
            Reporter: Jeremy Bauer
            Assignee: Jeremy Bauer


I posted this dev forum question regarding QuerySQLCache:

---
We've had a few regression-type issues (OPENJPA-660 & OPENJPA-731)
since the addition of QuerySQLCache in 1.2.0.  While this cache has
shown to provide significant performance improvements for certain
scenarios, I think we should consider disabling the cache by default.
The main reason is the potential for future regression issues.
Another reason is that we didn't follow the pattern set by existing
caches.  With the exception of the query compilation cache (which is
relatively static after queries are initialized), the other caches are
disabled by default.  While the cache is supposed to be transparent,
it could (and has shown to) affect the behavior of an application that
has been working for quite some time on a prior release.

Arguably, QuerySQLCache may just need more rigorous testing.  While
additional testing may help, testing every scenario is difficult and
if we miss one, regression issues due to performance enhancements are
pretty hard to swallow.  I think a better approach is to allow users
to enable the cache if they choose to do so.
---

And received positive responses.  The necessary updates will be made to disable the cache by default and committed to trunk.

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


[jira] Updated: (OPENJPA-748) Disable QuerySQLCache by default

Posted by "Donald Woods (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OPENJPA-748?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Donald Woods updated OPENJPA-748:
---------------------------------

    Fix Version/s:     (was: 2.0.0)
                   2.0.0-beta

> Disable QuerySQLCache by default
> --------------------------------
>
>                 Key: OPENJPA-748
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-748
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: jdbc
>    Affects Versions: 1.2.0, 1.3.0
>            Reporter: Jeremy Bauer
>            Assignee: Jeremy Bauer
>             Fix For: 2.0.0-beta
>
>
> I posted this dev forum question regarding QuerySQLCache:
> ---
> We've had a few regression-type issues (OPENJPA-660 & OPENJPA-731)
> since the addition of QuerySQLCache in 1.2.0.  While this cache has
> shown to provide significant performance improvements for certain
> scenarios, I think we should consider disabling the cache by default.
> The main reason is the potential for future regression issues.
> Another reason is that we didn't follow the pattern set by existing
> caches.  With the exception of the query compilation cache (which is
> relatively static after queries are initialized), the other caches are
> disabled by default.  While the cache is supposed to be transparent,
> it could (and has shown to) affect the behavior of an application that
> has been working for quite some time on a prior release.
> Arguably, QuerySQLCache may just need more rigorous testing.  While
> additional testing may help, testing every scenario is difficult and
> if we miss one, regression issues due to performance enhancements are
> pretty hard to swallow.  I think a better approach is to allow users
> to enable the cache if they choose to do so.
> ---
> And received positive responses.  The necessary updates will be made to disable the cache by default and committed to trunk.

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


[jira] Closed: (OPENJPA-748) Disable QuerySQLCache by default

Posted by "Jeremy Bauer (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OPENJPA-748?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jeremy Bauer closed OPENJPA-748.
--------------------------------

       Resolution: Fixed
    Fix Version/s: 2.0.0

No longer an issue since PreparedQueryCache replaced QuerySQLCache in 2.0.0.  Changing default behavior in 1.2 is unlikely.

> Disable QuerySQLCache by default
> --------------------------------
>
>                 Key: OPENJPA-748
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-748
>             Project: OpenJPA
>          Issue Type: Bug
>          Components: jdbc
>    Affects Versions: 1.2.0, 1.3.0
>            Reporter: Jeremy Bauer
>            Assignee: Jeremy Bauer
>             Fix For: 2.0.0
>
>
> I posted this dev forum question regarding QuerySQLCache:
> ---
> We've had a few regression-type issues (OPENJPA-660 & OPENJPA-731)
> since the addition of QuerySQLCache in 1.2.0.  While this cache has
> shown to provide significant performance improvements for certain
> scenarios, I think we should consider disabling the cache by default.
> The main reason is the potential for future regression issues.
> Another reason is that we didn't follow the pattern set by existing
> caches.  With the exception of the query compilation cache (which is
> relatively static after queries are initialized), the other caches are
> disabled by default.  While the cache is supposed to be transparent,
> it could (and has shown to) affect the behavior of an application that
> has been working for quite some time on a prior release.
> Arguably, QuerySQLCache may just need more rigorous testing.  While
> additional testing may help, testing every scenario is difficult and
> if we miss one, regression issues due to performance enhancements are
> pretty hard to swallow.  I think a better approach is to allow users
> to enable the cache if they choose to do so.
> ---
> And received positive responses.  The necessary updates will be made to disable the cache by default and committed to trunk.

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