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"

Task #10087 (new)

Opened 11 years ago

Last modified 8 years ago

Refactor sf.executor to handle exceptions

Reported by: jamoore Owned by:
Priority: minor Milestone: Unscheduled
Component: Services Version: n.a.
Keywords: lhf Cc: mtbcarroll, spli, bpindelski
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

See https://github.com/openmicroscopy/openmicroscopy/pull/575

Rather than wrapping each call to executor.execute with try {} catch { IceMapper....} the sf executor instance variable should be properly made private and a single execute method introduced which takes the Ice.Current instance and "does the right thing".

There may be other locations without access to a sf (i.e. the ServiceFactoryI ) in which case the execute method may need to go in a helper class rather than the ServiceFactoryI` calss.

Change History (1)

comment:1 Changed 8 years ago by jamoore

  • Milestone changed from 5.x to Unscheduled
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.73284 sec.)

We're Hiring!