tags:

views:

1016

answers:

4

Hello all,

I am getting the following exception when I try to update an object:

org.hibernate.TransientObjectException: object references an unsaved transient instance - save the transient instance before flushing: ......

Can anyone help???

The object that I am trying to update has the 'lazy' attribute set to false in the mapping file. Seems like hibernate is expecting me to save child objects before it flushes the update???

EDIT (ADDED):

<hibernate-mapping>
    <class name="utils.message.Message" table="messages">
     <id name="id" column="message_id">
      <generator class="native" />
     </id>
     <property name="message_text" column="message_text" />
     <property name="message_file" column="message_file" />
     <property name="is_active" column="is_active" type="boolean"/>
     <property name="is_global" column="is_global" type="boolean"/>
     <property name="start" column="start" type="java.util.Date"/>
     <property name="end" column="end" type="java.util.Date"/>
     <property name="last_updated" column="last_updated" type="java.util.Date"/>  

     <many-to-one name="last_updated_by" class="utils.user.User" column="last_updated_by" lazy="false"/>
     <many-to-one name="healthDepartment" class="utils.healthdepartment.HealthDepartment" column="health_department_id" lazy="false"/>

    </class>
</hibernate-mapping>
+3  A: 

TransientObjectException occurs when you save an object which references another object that is transient (meaning it has the "default" identifier value, frequently null) and then flush the Session. This commonly happens when you are creating an entire graph of new objects but haven't explicitly saved all of them. There are two ways to work around this:

  1. As you suggest, you could use cascading of saves to other associated objects. However, cascading wasn't really intended as a workaround for TOE but rather as a convenience for saving a group of related objects that are frequently manipulated together. If you detach your objects without its full set of associated data and then save it with cascading enabled, you could inadvertently delete data you don't want to lose.
  2. Ensure that all transient objects in your graph are explicitly saved as part of your unit of work. This is really just a case of understanding how your application will be creating an object graph and what entities are transient and which might be persistent or detached.

I would recommend reading this entire chapter from the Hibernate docs to understand fully the terminology of transient, persistent and detached:

http://docs.jboss.org/hibernate/stable/core/reference/en/html/objectstate.html

cliff.meyers
A: 

With a many-to-one relationship, it would not be appropriate for Hibernate to cascade persistence operations, since the "one" is conceptually an entity shared between many others. This kind of relationship isn't a "child object", as you put it.

You can override this behaviour by explicitly setting the cascade attribute on the relation, or you can manually persist the other end of the many-to-one relations.

skaffman
Skaffman,Thanks for the clarification. I just read the doc suggested by brd6644. It helped better understand some fundamental concepts of Hibernate.Ultimately, what I need some help figuring out is how to be able to update a Message without cascading down the operation to the User/HD associated with the Message.
rafael
A: 

App is in a Spring environment. Fix: to run update from within Hibernate environment.

rafael
A: 

dude use property "cascade = true" in mapping. all will well... V-Boy

vishu