openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rick Curtis (JIRA)" <>
Subject [jira] Updated: (OPENJPA-1855) OpenJPA shouldn't silently ignore an invalid javax.persistence.xxxx configuration property
Date Tue, 26 Oct 2010 18:36:23 GMT


Rick Curtis updated OPENJPA-1855:

    Attachment: OPENJPA-1855.patch

The change I'm proposing entails logging a WARNING message anytime "javax.persistence.cache.retrieveMode"
or "javax.persistence.cache.storeMode" in encountered. I went back and forth with TRACE vs
WARN and I think WARN is the correct direction. This can easily be extended to other javax.persistence.[xx]
properties that can't be configured at the provider level.

The message looks something like this : 

16  test  WARN   [main] openjpa.Runtime - The property "javax.persistence.cache.storeMode=BYPASS"
was detected while loading configuration. However, it is invalid and cannot be configured
at the provider level, so it is ignored. Please consult the documentation for the correct
usage of this property.

> OpenJPA shouldn't silently ignore an invalid javax.persistence.xxxx configuration property
> ------------------------------------------------------------------------------------------
>                 Key: OPENJPA-1855
>                 URL:
>             Project: OpenJPA
>          Issue Type: Improvement
>          Components: logging
>            Reporter: Rick Curtis
>            Assignee: Rick Curtis
>            Priority: Minor
>         Attachments: OPENJPA-1855.patch
> While I was working on a unit test I had the following property set and it was being
silently ignored by the runtime. After digging around a bunch, I figured out that it is actually
an invalid configuration. I can specify this property on em.setProperty(...)[3.7.2], but I
can't set a 'default' value at the provider(emf) level.
> <property name="javax.persistence.cache.storeMode" value="BYPASS"/>
> [3.7.2] -
> Cache retrieve mode and cache store mode properties may be specified at the level of
the persistence
> context by means of the EntityManager setProperty method. These properties may be specified
> the EntityManager find and refresh methods and the Query and TypedQuery setHint methods.
> Cache retrieve mode and/or cache store mode properties specified for the find, refresh,
> Query and TypedQuery setHint methods override those specified for the persistence context
> the specified find and refresh invocations, and for the execution of the specified queries
> With this JIRA I want to detect *some* of the invalid configurations and log a .... [INFO,WARN,TRACE]?
message. Per the spec ( "If a persistence provider does not recognize a property (other
than a property defined by this specification), the provider must ignore it." I believe this
gives us the leeway to do something about the condition that I ran into.

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

View raw message