jpa problem

[vik ar]

Hi. I am getting the following error

Caused by: org.hibernate.PropertyValueException: not-null property references a null or transient value: com.ao.persistenceservice.domainobjects.Person.address
	at org.hibernate.engine.Nullability.checkNullability(Nullability.java:72)
	at org.hibernate.event.def.AbstractSaveEventListener.performSaveOrReplicate(AbstractSaveEventListener.java:284)
	at org.hibernate.event.def.AbstractSaveEventListener.performSave(AbstractSaveEventListener.java:180)
	at org.hibernate.event.def.AbstractSaveEventListener.saveWithGeneratedId(AbstractSaveEventListener.java:121)
	at org.hibernate.event.def.DefaultPersistEventListener.entityIsTransient(DefaultPersistEventListener.java:131)
	at org.hibernate.event.def.DefaultPersistEventListener.onPersist(DefaultPersistEventListener.java:87)
	at org.hibernate.event.def.DefaultPersistEventListener.onPersist(DefaultPersistEventListener.java:38)
	at org.hibernate.impl.SessionImpl.firePersist(SessionImpl.java:618)
	at org.hibernate.impl.SessionImpl.persist(SessionImpl.java:592)
	at org.hibernate.impl.SessionImpl.persist(SessionImpl.java:596)
	at org.hibernate.ejb.AbstractEntityManagerImpl.persist(AbstractEntityManagerImpl.java:212)
	... 39 more

I have made sure in debug mode the following persist code is functioning as required.

 Address address = new Address(addressRequest.getPostcode());
    	for (String name :personRequest.getName()){
    		Person person = new Person(address,name); 
    		em.persist(person);
    	}

My entity classes are as follows:

Person.java

package com.ao.persistenceservice.domainobjects;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.ManyToOne;
import javax.persistence.SequenceGenerator;
import javax.persistence.Table;

/**
 * Person generated by hbm2java
 */
@Entity
@Table(name="PERSON"
)
public class Person  implements java.io.Serializable {

    // Fields    

     private long personid;
     private Address address;
     private String name;

     // Constructors

    /** default constructor */
    public Person() {
    }

    /** default constructor */
    public Person(String name) {
    	this.name = name;
    }
    
    /** full constructor */
    public Person(Address address, String name) {
       this.address = address;
       this.name = name;
    }
   
    // Property accessors
     @Id
     @GeneratedValue(generator = "PersonSeq", 
             strategy = GenerationType.SEQUENCE)
             @SequenceGenerator(name = "PersonSeq", 
                                sequenceName = "seq_person_id", 
                                allocationSize = 1)
    @Column(name="PERSONID", unique=true, nullable=false, insertable=true, updatable=true, precision=11, scale=0)
    public long getPersonid() {
        return this.personid;
    }
    
    public void setPersonid(long personid) {
        this.personid = personid;
    }
@ManyToOne(cascade={},
        fetch=FetchType.LAZY)
    
    @JoinColumn(name="ADDRESSID", unique=false, nullable=false, insertable=true, updatable=true)
    public Address getAddress() {
        return this.address;
    }
    
    public void setAddress(Address address) {
        this.address = address;
    }
    
    @Column(name="NAME", unique=false, nullable=true, insertable=true, updatable=true, length=100)
    public String getName() {
        return this.name;
    }
    
    public void setName(String name) {
        this.name = name;
    }
}

Address.java

package com.ao.persistenceservice.domainobjects;

import java.util.HashSet;
import java.util.Set;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.OneToMany;
import javax.persistence.SequenceGenerator;
import javax.persistence.Table;

/**
 * Address generated by hbm2java
 */
@Entity
@Table(name="ADDRESS"
)
public class Address  implements java.io.Serializable {


     private long addressid;
     private String postcode;
     private Set<Person> persons = new HashSet<Person>(0);

    public Address() {
    }

    public Address(String postcode) {
       this.postcode = postcode;
    }
   
     @Id 
     @GeneratedValue(generator = "AddressSeq", 
             strategy = GenerationType.SEQUENCE)
             @SequenceGenerator(name = "AddressSeq", 
                                sequenceName = "seq_address_id", 
                                allocationSize = 1)
    @Column(name="ADDRESSID", unique=true, nullable=false, precision=11, scale=0)
    public long getAddressid() {
        return this.addressid;
    }
    
    public void setAddressid(long addressid) {
        this.addressid = addressid;
    }

    
    @Column(name="POSTCODE", length=100)
    public String getPostcode() {
        return this.postcode;
    }
    
    public void setPostcode(String postcode) {
        this.postcode = postcode;
    }

@OneToMany(fetch=FetchType.LAZY, mappedBy="address")
    public Set<Person> getPersons() {
        return this.persons;
    }
    
    public void setPersons(Set<Person> persons) {
        this.persons = persons;
    }
}

[vik ar]

I have reworked my code as follows:

persistence class

Set<Person> persons = new HashSet<Person>();
    	for (String name :personRequest.getName()){
    		Person person = new Person(name); 
    		persons.add(person);
    	}
    	Address address = new Address(addressRequest.getPostcode(),persons);
    	em.persist(address);    

Address.java

package com.ao.persistenceservice.domainobjects;

import java.util.HashSet;
import java.util.Set;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.OneToMany;
import javax.persistence.SequenceGenerator;
import javax.persistence.Table;

/**
 * Address generated by hbm2java
 */
@Entity
@Table(name="ADDRESS"
)
public class Address  implements java.io.Serializable {


     private long addressid;
     private String postcode;
     private Set<Person> persons = new HashSet<Person>(0);

    public Address() {
    }

    public Address(String postcode,Set<Person> persons) {
       this.postcode = postcode;
    }
   
     @Id 
     @GeneratedValue(generator = "AddressSeq", 
             strategy = GenerationType.SEQUENCE)
             @SequenceGenerator(name = "AddressSeq", 
                                sequenceName = "seq_address_id", 
                                allocationSize = 1)
    @Column(name="ADDRESSID", unique=true, nullable=false, precision=11, scale=0)
    public long getAddressid() {
        return this.addressid;
    }
    
    public void setAddressid(long addressid) {
        this.addressid = addressid;
    }

    
    @Column(name="POSTCODE", length=100)
    public String getPostcode() {
        return this.postcode;
    }
    
    public void setPostcode(String postcode) {
        this.postcode = postcode;
    }

@OneToMany(fetch=FetchType.EAGER)
@JoinColumn(name="ADDRESSID")
    public Set<Person> getPersons() {
        return this.persons;
    }
    
    public void setPersons(Set<Person> persons) {
        this.persons = persons;
    }
}

Person.java

package com.ao.persistenceservice.domainobjects;

import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.FetchType;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.JoinColumn;
import javax.persistence.ManyToOne;
import javax.persistence.SequenceGenerator;
import javax.persistence.Table;

/**
 * Person generated by hbm2java
 */
@Entity
@Table(name="PERSON"
)
public class Person  implements java.io.Serializable {

    // Fields    

     private long personid;
     private String name;

     // Constructors

    /** default constructor */
    public Person() {
    }
    
    /** full constructor */
    public Person(String name) {
       this.name = name;
    }
   
    // Property accessors
     @Id
     @GeneratedValue(generator = "PersonSeq", 
             strategy = GenerationType.SEQUENCE)
             @SequenceGenerator(name = "PersonSeq", 
                                sequenceName = "seq_person_id", 
                                allocationSize = 1)
    @Column(name="PERSONID", unique=true, nullable=false, insertable=true, updatable=true, precision=11, scale=0)
    public long getPersonid() {
        return this.personid;
    }
    
    public void setPersonid(long personid) {
        this.personid = personid;
    }
/*@ManyToOne(cascade={},
        fetch=FetchType.LAZY)
    
    @JoinColumn(name="ADDRESSID", unique=false, nullable=false, insertable=true, updatable=true)
    public Address getAddress() {
        return this.address;
    }
    
    public void setAddress(Address address) {
        this.address = address;
    }*/
    
    @Column(name="NAME", unique=false, nullable=true, insertable=true, updatable=true, length=100)
    public String getName() {
        return this.name;
    }
    
    public void setName(String name) {
        this.name = name;
    }
}
   

I am getting the error

java.sql.BatchUpdateException: ORA-00942: table or view does not exist

My datasource is connecting to oracle as the system administrator and the tables Person and Address are in the schema rbv. I am wondering if this is the issue?

[vik ar]

right in the classes I prefixed the table names with 'rbv.' My address table is being populated but the person table is empty. Do I need to save the person objects seperately??? I thought they would be persisted by simply persisting my address object???

[vik ar]

Address.java reworked

@OneToMany(fetch=FetchType.EAGER,cascade = CascadeType.ALL)  
 @JoinColumn(name="ADDRESSID")  
     public Set<Person> getPersons() {  
         return this.persons;  
     }     

Person table is still empty?

[Himanshu Chitranshi]

i think the user id and password which you are using in obtaining the connection with database do not have sufficient permission.

Use the same user and password and login to oracle and fire any update or insert query..check whether you are getting the same ORA error message....

[vik ar]

I am not getting an ORA error message at the point of persist. If it is a permission problem how come the address table is being populated fine? Unless there is some special permission required for the cascade which I dont know about. My sql is as follows

connect system/password;
drop user rbv cascade;
create user rbv identified by password;
grant connect, create session, create table, insert any table, create sequence, create trigger, unlimited tablespace to rbv;
connect rbv/password;
CREATE TABLE address ( addressId number(11) NOT NULL primary key, postcode varchar2(100));
CREATE TABLE person ( personId number(11) NOT NULL primary key, name varchar2(100), addressId number(11) NOT NULL REFERENCES address);
create sequence seq_person_id
start with 1
increment by 1; 
create sequence seq_address_id
start with 1
increment by 1; 
create or replace trigger trig_person 
before insert on person 
for each row 
begin 
IF :NEW.personId IS NULL THEN
select seq_person_id.nextval into :new.personId from dual;
END IF;
end; 
/
create or replace trigger trig_address 
before insert on address 
for each row 
begin 
IF :NEW.addressId IS NULL THEN
select seq_address_id.nextval into :new.addressId from dual;
END IF;
end;
/

After reworking my sql as above and changing my datasource to connect as the rbv user I no longer need to prefix the table names in the classes with rbv. However the cascade still doesnt work?

[Himanshu Chitranshi]

Hello,

In the @Table annotation can you please use the schema attribute explicitly..

I mean in person use

@Table(name="PERSON",schema="rbv")

In Address use

@Table(name="ADDRESS",schema="rbv")

Default value of schema is "" so may be it is creating the problem..