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"

Task #263 (closed)

Opened 14 years ago

Closed 8 years ago

Unloaded collections/links are a pain

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: GatherReqs
Component: Documentation Version: 3.0-M3
Keywords: links,collections,unloading Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.


Writing this one up under documentation. Am also adding documentation to [ Omero+Object+Model] on the tiki.

As seen in:

the fact that collections can get unloaded (set to null) or filtered (have their name add to Details.filteredSet()) is very difficult for users. Calls to link<Class> or unlink<Class> fail mysteriously. The simplest solution is to take the checks in the model classes a step further (this is actually the defect here) and check filtered as well as null.

All calls no matter what throw an exception. This then needs to be documented in the API calls that the graphs that are returned "will throw an IllegalStateException when someObject.getSomeOtherObject() is called". Users of the generic IUpdate and IQuery should be equally forewarned.

The only alternative really is to not lazily load our collections. This isn't too bad until the case comes that a dataset with 10,000 images should be returned, but the user doesn't care about the images.

More sensible is to clarify the add<Class>/link<Class>/etc. model API and to make the exceptions as understandable as possible. (In the special case of links, #208 is also important.)

Change History (6)

comment:1 Changed 14 years ago by jmoore

  • Keywords iteration6 added

comment:2 Changed 14 years ago by jmoore

  • Milestone changed from 3.0-M3 to GatherReqs

comment:3 Changed 14 years ago by jmoore

  • Keywords iteration6 removed

comment:4 Changed 14 years ago by jmoore

Note also need to examine the difference between what the read filter does and manual filtering. Read filter does not_ update the filtered set!

comment:5 Changed 8 years ago by hflynn

This is covered in the developer docs these days, isn't it? I'm sure I saw something along these lines while editing recently. Can we close this?

comment:6 Changed 8 years ago by jmoore

  • Resolution set to invalid
  • Status changed from new to closed

Likely the state of the code isn't as well-documented as it should be, nor is the state of the docs as up-to-date as it should be, but there isn't a concrete TODO here other than just going through all the docs, so happy to close.

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.155814 sec.)

We're Hiring!