openjpa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rafael Teixeira (JIRA)" <>
Subject [jira] [Created] (OPENJPA-2463) Wrong logging level on message issued by the fix to OPENJPA-2233
Date Wed, 04 Dec 2013 15:55:35 GMT
Rafael Teixeira created OPENJPA-2463:

             Summary: Wrong logging level on message issued by the fix to OPENJPA-2233
                 Key: OPENJPA-2463
             Project: OpenJPA
          Issue Type: Bug
          Components: Enhance
    Affects Versions: 2.2.0, 2.1.0, 2.0.1
         Environment: OpenJPA as bundled on WebSphere 8.5.5
            Reporter: Rafael Teixeira
            Priority: Minor

Patch for issue OPENJPA-2233 introduced a message when a Embeddable entity defines an ID property,
which the Enhancer can't properly add a helper method.

--- openjpa-kernel/src/main/java/org/apache/openjpa/enhance/	(revision 1325823)
+++ openjpa-kernel/src/main/java/org/apache/openjpa/enhance/	(working copy)
@@ -1150,6 +1150,11 @@
             if (_meta.hasAbstractPKField() == true) { 
+        	if (_meta.isEmbeddedOnly() 
+            		&& _meta.getIdentityType() == ClassMetaData.ID_APPLICATION) {
+            	_log.error(_loc.get("ID-field-in-embeddable-unsupported", _meta.toString()));
+            }

The problem is this message is also emitted when inheriting from a MappedSuperClass, which
is perfectly normal to have an Id field/property. As OpenJPA works perfectly with such a class
hierarchy (And even the Enhancer with the complement of that patch just skips adding a call
to the non-added helper method), it is pointless to alarm operations personnel with a message
labeled and tagged as an error.

Therefore I suggest to log it as a warning, also editing the corresponding message text to
better indicate that the Enhancer will not add/call the helper method.

This message was sent by Atlassian JIRA

View raw message