Hibernate Spring problem with Illegal attempt to associate a collection with two open sessions

[Carlos Gosu]

Hello,

I'm having problems trying to update an object with a collection associated that was previously retrieved. If someone has faced the problem before it would be a great help.

The simplified code is the following:

<hibernate-mapping>
	<class name="persistence.bean.DiselPBeanMessage" table="MESSAGE" >
		<id name="idMessage" type="long">
			<column name="IDMESSAGE" precision="10" scale="0" />
			<generator class="sequence">
				<param name="sequence">S_MESSAGE</param>
			</generator>
		</id>
		<property name="idReceiver" type="java.lang.Long">
			<column name="IDRECEIVER" precision="10" scale="0" />
		</property>
		<many-to-one name="receiver" class="persistence.bean.PBeanReceiver" update="false" insert="false" fetch="join">
			<column name="IDRECEIVER" precision="10" scale="0" />
		</many-to-one>
	</class>
</hibernate-mapping>

<hibernate-mapping>
	<class name="persistence.bean.PBeanReceiver" table="RECEIVER" >
		<id name="idReceiver" type="long">
			<column name="IDAMBITOPRESCRIPCION" precision="10" scale="0" />
			<generator class="sequence">
				<param name="sequence">S_RECEIVER</param>
			</generator>
		</id>
		<property name="name" type="string">
			<column name="NAME" length="50" not-null="true" />
		</property>
		
		<set name="messages" inverse="true">
			<key>
				<column name="IDRECEIVER" precision="10" scale="0" />
			</key>
			<one-to-many class="persistence.bean.PBeanMessage" />
		</set>
	</class>
</hibernate-mapping>

Basicly the code that produces the exception is:

DiselPBeanMessage message = dao.loadMenssageById("82");  //Implemented with getSession().createCriteria(DiselPBeanMessage.class) in a class which extends HibernateDaoSupport
message.getReceiver().setName("Myself");
dao.saveMessage(message);

The implementation of saveMessage is:

public void saveMessage(PBeanMessage message) {
		try {
			PBeanReceiver receiver = message.getReceiver();

			if (receiver != null) {
				getHibernateTemplate.saveOrUpdate(receiver);
				message.setIdReceiver(receiver.getIdReceiver());
			}

			getHibernateTemplate.saveOrUpdate(message);
		} catch (HibernateException e) {
			throw getHibernateTemplate().convertHibernateAccessException(e);
		}
	}

I got the so called Illegal attempt to associate a collection with two open sessions in the getHibernateTemplate.saveOrUpdate(receiver); sentence. Any idea?

By the way I read around that it would be necessary to evict the object but I shouldn't evict the object out of the saveMessage for design reasons.

[Vijitha Kumara]

Can you post the stack trace here?

[Carlos Gosu]

The stack trace is the following:

GRAVE: org.hibernate.HibernateException: Illegal attempt to associate
a collection with two open sessions
at
org.hibernate.collection.AbstractPersistentCollection.setCurrentSession(AbstractPersistentCollection.java:
410)
at
org.hibernate.event.def.OnUpdateVisitor.processCollection(OnUpdateVisitor.java:
43)
at
org.hibernate.event.def.AbstractVisitor.processValue(AbstractVisitor.java:
101)
at
org.hibernate.event.def.AbstractVisitor.processValue(AbstractVisitor.java:
61)
at
org.hibernate.event.def.AbstractVisitor.processEntityPropertyValues(AbstractVisitor.java:
55)
at
org.hibernate.event.def.AbstractVisitor.process(AbstractVisitor.java:
123)
at
org.hibernate.event.def.DefaultSaveOrUpdateEventListener.performUpdate(DefaultSaveOrUpdateEventListener.java:
293)
at
org.hibernate.event.def.DefaultSaveOrUpdateEventListener.entityIsDetached(DefaultSaveOrUpdateEventListener.java:
223)
at
org.hibernate.event.def.DefaultUpdateEventListener.performSaveOrUpdate(DefaultUpdateEventListener.java:
33)
at
org.hibernate.event.def.DefaultSaveOrUpdateEventListener.onSaveOrUpdate(DefaultSaveOrUpdateEventListener.java:
70)

[Amjad Aziz]

I have same problem, and got fixed by putting @Transactional annotation.
Can you try

@Transactional
public void saveMessage(PBeanMessage message) {...}

[J van der Linde]

Amjad Aziz wrote:I have same problem, and got fixed by putting @Transactional annotation.
Can you try

@Transactional
public void saveMessage(PBeanMessage message) {...}

This worked perfectly, thanks Aziz!