Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Bug #61 (closed)

Opened 18 years ago

Closed 18 years ago

Hibernate "PersistenceSet" instances are escaping.

Reported by: jamoore Owned by: jamoore
Priority: major Cc:
Sprint: n.a.
Total Remaining Time: n.a.

Description

ProxyCleanupFilter? removes all non-initialized Hibernate proxies (instances of
PeristenceSet? for collections) but does not replace all sets with java.util.*
instances. It must, otherwise the client must have hibernate.jar on the classpath.

Additional Comment #1 From Josh Moore 2006-05-02 09:00

[r621] Wrapping Hibernate implementations of Set and List in HashSet? and ArrayList?

  • Not most performant solution; for that, use own (ome.util.*) wrapper class
  • Not catching everything; see PersistentMap? et al.

[r657] Caught another aspect of this : The arrays that were being passed out.

Originally : http://bugs.openmicroscopy.org.uk/show_bug.cgi?id=647

Change History (3)

comment:1 Changed 18 years ago by jmoore

  • Milestone changed from 3.0-M2 to 3.0-M3
  • Priority changed from minor to major
  • Status changed from new to assigned

We should add arrays to the Filter interface along with anything else that can be returned. Then this can be closed.

comment:2 Changed 18 years ago by jmoore

  • Keywords set to iteration6

comment:3 Changed 18 years ago by jmoore

  • Resolution set to fixed
  • Status changed from assigned to closed

r986 adds a test for various Hibernate AbstractPersistentCollection subclasses (even ones we are not using). ProxyCleanupFilter is now handling PersistentMap properly. Closing.

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.69576 sec.)

We're Hiring!