[hibernate-issues] [Hibernate-JIRA] Commented: (HHH-5740) EXTRA Lazy collection with inverse owner: PersistenSet still contains previosly removed elements

EXTRA Lazy collection with inverse owner: PersistenSet still contains previosly removed elements ------------------------------------------------------------------------------------------------- Key: HHH-5740 URL: http://opensource.atlassian.com/projects/hibernate/browse/HHH-5740 Project: Hibernate Core Issue Type: Bug Components: core Affects Versions: 3.5.0-Final, 3.6.0 Environment: Hibernate 3.6.0 (also 3.5.0 is affect of this bug), database indipendent (attached Testcase is based on hsqldb) Reporter: Guenther Demetz Priority: Critical Attachments: TestExtraLazyCollectionWithInverseOwner.jar When mapping a Set with inverse owner (= specifying mappedBy attribute) and annotated with @org.hibernate.annotations.LazyCollection (LazyCollectionOption.EXTRA) then the removal of single elements doesn't work as expected: the PersistentSet still contains elements which have previously removed. .. assertTrue(a.assD.remove(d)); // removing element d from PersistentSet, this assert goes well assertFalse(a.assD.contains(d)); // asserting PersistentSet doesn't contain element d anymore, FAILS !! The contains method implementation of PersistentSet properly calls flush() before executing the select-query, but anyway for some reason the registered DelayedOperation (as before enqueued by the remove call) isn't properly brought to execution if the collection is declared with inverse owner. This IMHO is a critical bug, as this behavior clearly violates the contracts of method java.util.Set#remove(java.lang.Object) where the specifications says: <quoted: "the set will not contain the specified element once the call returns"> Furthermore it could create serious damage to companies using hibernate: imagine for example a company cancels a order-position and nevertheless the canceled position is subsequently delivered and invoiced to the customer. Please see attached testcase for more details. Thanks for attention.