You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openjpa.apache.org by "DORE (JIRA)" <ji...@apache.org> on 2015/11/09 16:32:10 UTC

[jira] [Created] (OPENJPA-2613) findByPrimaryKey error on EJB CMP with composite primary key

DORE created OPENJPA-2613:
-----------------------------

             Summary: findByPrimaryKey error on EJB CMP with composite primary key
                 Key: OPENJPA-2613
                 URL: https://issues.apache.org/jira/browse/OPENJPA-2613
             Project: OpenJPA
          Issue Type: Bug
          Components: jpa, kernel
    Affects Versions: 2.4.0
            Reporter: DORE


Hello Openjpa community,
I'm trying to get working a cmp ejb entity with a composite primary key.
I know I can get this sample running with a simple Integer type as primary key, but some of cmp ejb have composite pk, so I want to test it. (not just for fun, I have some ejb oin this case !)
create, findAll, customer finders are ok, but I encounter problem on findByPrimaryKey:

{code}
java.rmi.RemoteException: The bean encountered a non-application exception; nested exception is: 
	<openjpa-2.4.0-r422266:1674604 nonfatal user error> org.apache.openjpa.persistence.ArgumentException: The given value "fr.chronopost.ejb.esd.customer.interfaces.CustomerPK@1" cannot be converted into an identity for "class openejb.fr.chronopost.ejb.esd.customer.server.CUSTOMER".  The value is the wrong type (fr.chronopost.ejb.esd.customer.interfaces.CustomerPK).
	at org.apache.openejb.core.transaction.EjbTransactionUtil.handleSystemException(EjbTransactionUtil.java:155)
	at org.apache.openejb.core.cmp.CmpContainer.findByPrimaryKey(CmpContainer.java:702)
	at org.apache.openejb.core.cmp.CmpContainer.invoke(CmpContainer.java:269)
	at org.apache.openejb.server.ejbd.EjbRequestHandler.doEjbHome_FIND(EjbRequestHandler.java:445)
	at org.apache.openejb.server.ejbd.EjbRequestHandler.processRequest(EjbRequestHandler.java:198)
	at org.apache.openejb.server.ejbd.EjbDaemon.processEjbRequest(EjbDaemon.java:344)
	at org.apache.openejb.server.ejbd.EjbDaemon.service(EjbDaemon.java:240)
	at org.apache.openejb.server.ejbd.EjbServer.service(EjbServer.java:86)
	at org.apache.openejb.server.httpd.ServerServlet.service(ServerServlet.java:58)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:731)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:220)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:122)
	at org.apache.tomee.catalina.OpenEJBValve.invoke(OpenEJBValve.java:44)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:170)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:103)
	at org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:957)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:116)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:423)
	at org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1079)
	at org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:620)
	at org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:318)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:895)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:918)
	at org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
	at java.lang.Thread.run(Thread.java:662)
Caused by: <openjpa-2.4.0-r422266:1674604 nonfatal user error> org.apache.openjpa.persistence.ArgumentException: The given value "fr.chronopost.ejb.esd.customer.interfaces.CustomerPK@1" cannot be converted into an identity for "class openejb.fr.chronopost.ejb.esd.customer.server.CUSTOMER".  The value is the wrong type (fr.chronopost.ejb.esd.customer.interfaces.CustomerPK).
	at org.apache.openjpa.kernel.BrokerImpl.newObjectId(BrokerImpl.java:1311)
	at org.apache.openjpa.kernel.DelegatingBroker.newObjectId(DelegatingBroker.java:315)
	at org.apache.openjpa.persistence.EntityManagerImpl.find(EntityManagerImpl.java:485)
	at org.apache.openejb.persistence.JtaEntityManager.find(JtaEntityManager.java:180)
	at org.apache.openejb.core.cmp.jpa.JpaCmpEngine.loadBean(JpaCmpEngine.java:163)
	at org.apache.openejb.core.cmp.CmpContainer.findByPrimaryKey(CmpContainer.java:688)
	... 25 more
Caused by: java.lang.Exception: java.lang.ClassCastException: fr.chronopost.ejb.esd.customer.interfaces.CustomerPK cannot be cast to java.lang.Number
	at org.apache.openjpa.util.Exceptions.replaceNestedThrowables(Exceptions.java:254)
	at org.apache.openjpa.persistence.ArgumentException.writeObject(ArgumentException.java:106)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
	at java.lang.reflect.Method.invoke(Method.java:597)
	at java.io.ObjectStreamClass.invokeWriteObject(ObjectStreamClass.java:940)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1469)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1400)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1158)
	at java.io.ObjectOutputStream.defaultWriteFields(ObjectOutputStream.java:1518)
	at java.io.ObjectOutputStream.writeSerialData(ObjectOutputStream.java:1483)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1400)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1158)
	at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:330)
	at org.apache.openejb.client.ThrowableArtifact.writeExternal(ThrowableArtifact.java:57)
	at java.io.ObjectOutputStream.writeExternalData(ObjectOutputStream.java:1429)
	at java.io.ObjectOutputStream.writeOrdinaryObject(ObjectOutputStream.java:1398)
	at java.io.ObjectOutputStream.writeObject0(ObjectOutputStream.java:1158)
	at java.io.ObjectOutputStream.writeObject(ObjectOutputStream.java:330)
	at org.apache.openejb.client.EJBResponse.writeExternal(EJBResponse.java:186)
	at org.apache.openejb.server.ejbd.EjbRequestHandler.processResponse(EjbRequestHandler.java:303)
	at org.apache.openejb.server.ejbd.EjbDaemon.processEjbResponse(EjbDaemon.java:356)
	at org.apache.openejb.server.ejbd.EjbDaemon.service(EjbDaemon.java:269)
	... 20 more
{code}

{code:title=ejb-jar.xml|borderStyle=solid}
<enterprise-beans>
	<entity>
		<description>
			This bean represents customer
		</description>
		<ejb-name>CustomerEJB</ejb-name>
		<home>fr.chronopost.ejb.esd.customer.interfaces.CustomerHome</home>
		<remote>fr.chronopost.ejb.esd.customer.interfaces.CustomerRemote</remote>
		<ejb-class>fr.chronopost.ejb.esd.customer.server.CustomerBean</ejb-class>
		<persistence-type>Container</persistence-type>
		<prim-key-class>fr.chronopost.ejb.esd.customer.interfaces.CustomerPK</prim-key-class>
		<reentrant>false</reentrant>
		<cmp-version>2.x</cmp-version>
		<abstract-schema-name>CUSTOMER</abstract-schema-name>

		<cmp-field>
			<field-name>id</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>name</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>birthdate</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>sssNo</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>address</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>annualSalary</field-name>
		</cmp-field>
		<cmp-field>
			<field-name>loanAmount</field-name>
		</cmp-field>

		<primkey-field>id</primkey-field> 

		<query>
			<query-method>
				<method-name>findBySssNo</method-name>
				<method-params>
					<method-param>java.lang.String</method-param>
				</method-params>
			</query-method>
			<ejb-ql>select distinct object(o) from CUSTOMER o where o.sssNo = ?1</ejb-ql>
		</query>
		<query>
			<query-method>
				<method-name>findAll</method-name>
				<method-params />
			</query-method>
			<ejb-ql>select distinct object(o) from CUSTOMER o</ejb-ql>
		</query>
	</entity>
</enterprise-beans>
</ejb-jar>
{code}


{code:title=openejb-jar.xml|borderStyle=solid}
<?xml version="1.0" encoding="UTF-8"?>
<openejb-jar xmlns="http://www.openejb.org/xml/ns/openejb-jar-2.1"
	xmlns:nam="http://geronimo.apache.org/xml/ns/naming-1.1" xmlns:pkgen="http//www.openejb.org/xml/ns/pkgen-2.0"
	xmlns:sec="http://geronimo.apache.org/xml/ns/security-1.1" xmlns:sys="http://geronimo.apache.org/xml/ns/deployment-1.1">

	<cmp-connection-factory>
		<resource-link>ejbPool_esd</resource-link>

	</cmp-connection-factory>

	<enterprise-beans>

		<entity>
			<ejb-name>CustomerEJB</ejb-name>

         
			<table-name>CUSTOMER</table-name>
			<cmp-field-mapping>
				<cmp-field-name>id</cmp-field-name>
				<table-column>id</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>name</cmp-field-name>
				<table-column>name</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>birthdate</cmp-field-name>
				<table-column>birthdate</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>sssNo</cmp-field-name>
				<table-column>sss_no</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>address</cmp-field-name>
				<table-column>address</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>annualSalary</cmp-field-name>
				<table-column>annual_salary</table-column>
			</cmp-field-mapping>
			<cmp-field-mapping>
				<cmp-field-name>loanAmount</cmp-field-name>
				<table-column>loan_amount</table-column>
			</cmp-field-mapping>

			<query>
				<query-method>
					<method-name>findBySssNo</method-name>
					<method-params>
						<method-param>java.lang.String</method-param>
					</method-params>					
				</query-method>
				<ejb-ql>SELECT OBJECT(c) FROM CUSTOMER as c where c.sssNo = ?1</ejb-ql>
			</query>

			<query>
				<query-method>
					<method-name>findAll</method-name>
					<method-params />
				</query-method>
				<ejb-ql>SELECT OBJECT(c) FROM CUSTOMER as c</ejb-ql>
			</query>
		</entity>
	</enterprise-beans>
</openejb-jar>
{code}

{code:title=CustomerPK.java|borderStyle=solid}
public class CustomerPK implements java.io.Serializable, Comparable {

	public Integer id;  
	
	public CustomerPK() {
	}

	public CustomerPK(Integer id) {
		 this.id = id;
	}
	 
	public boolean equals(Object obj) {
		boolean ret = false;

		if (obj == null || !(obj instanceof CustomerPK))
			ret = false;
		else if (obj == this)
			ret = true;
		else if ((((CustomerPK) obj).id).compareTo(id) == 0)
			ret = true;

		return ret;
	}

	public int hashCode() {
		return id.hashCode();
	}

	public int compareTo(Object obj) {
		return ((CustomerPK) obj).id.compareTo(id);
	}
}
{code}

{code:title=CustomerRemote.java|borderStyle=solid}
public interface CustomerRemote extends EJBObject {
	
	public void setName(String name) throws RemoteException;
	public String getName() throws RemoteException;
	
	public void setSssNo(String sssNo) throws RemoteException;
	public String getSssNo() throws RemoteException;
	
	public void setAddress(String address) throws RemoteException;
	public String getAddress() throws RemoteException;
	
	public void setBirthdate(Date birthdate) throws RemoteException;
	public Date getBirthdate() throws RemoteException;
	
	public void setAnnualSalary(Double annualSalary) throws RemoteException;
	public Double getAnnualSalary() throws RemoteException;
	
	public void setLoanAmount(Double loanAmount) throws RemoteException;
	public Double getLoanAmount() throws RemoteException;
}
{code}

{code:title=CustomerRemote.java|borderStyle=solid}
public interface CustomerHome extends EJBHome {
	public CustomerRemote create(Integer id,String name,Date birthdate,String sssNo,String address, Double annualSalary,Double loanAmount) throws CreateException,RemoteException;
	
	public CustomerRemote create(CustomerPK primaryKey)throws CreateException,RemoteException;

	public CustomerRemote findByPrimaryKey(CustomerPK pk) throws FinderException, RemoteException;

	public Collection findAll() throws FinderException, RemoteException;

	public CustomerRemote findBySssNo(String sssNo) throws FinderException, RemoteException;
}
{code}


{code:title=...and here the test sample|borderStyle=solid}
...
CustomerHome home = ...
...
// create is OK
home.create(new java.lang.Integer(11), "name", new java.util.Date(), "123", "address", new Double(2.0), new Double(2.0));			

// findAll is OK
Collection col = home.findAll(); 
System.out.println("col.size=" + col.size());
for (Iterator iterator = col.iterator(); iterator.hasNext();) {
	Object object = (Object) iterator.next();
	CustomerRemote remote = (CustomerRemote) object;
	System.out.println(remote.getPrimaryKey() + " // " +  remote.getName() + " / " + remote.getAddress());
			}
// customer finder is OK too
CustomerRemote cr =  home.findBySssNo("123");
System.out.println(cr.getPrimaryKey() + " // " + cr.getName() + " / " + cr.getAddress());

// but findByPrimaryKey throws an exception !! (see stacktrace)
CustomerPK pk0 = new CustomerPK();
pk0.id = new Integer(11);
CustomerRemote remote =  home.findByPrimaryKey(pk0);
System.out.println(remote.getPrimaryKey() + " // " + remote.getName() + " / " + remote.getAddress());
...
{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)