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 #5549 (closed)

Opened 12 years ago

Closed 12 years ago

Bug: LockTimeout can also be thrown like MissingPyramidException

Reported by: jamoore Owned by: wmoore
Priority: blocker Milestone: OMERO-Beta4.3
Component: Services Version: n.a.
Keywords: n.a. Cc: cxallan, atarkowska, wmoore, jburel
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-06-16 (14)

Description

When the writing of the pyramid is actually happening, a LockTimeout can be thrown instead of a MissingPyramidException. LockTimeout is another ConcurrencyException, and so can be handled identically. Should we change the exception handlers to use that base class?

See attached log.

Attachments (1)

locktimeout.txt (20.8 KB) - added by jmoore 12 years ago.

Download all attachments as: .zip

Change History (10)

Changed 12 years ago by jmoore

comment:1 Changed 12 years ago by jburel

Having two exceptions to indicate that the pyramid is not ready is probably not the best approach
The result being the same: the pyramid is not ready so the image cannot be viewed.
We should only use one, the message will be different

comment:2 Changed 12 years ago by jburel

  • Owner set to jburel
  • Remaining Time set to 0.15
  • Status changed from new to accepted

Picking for insight, I will pass it Ola or Will when i am done.

comment:3 Changed 12 years ago by wmoore

Hi JM - might be better for Chris to handle this, since he's been doing the other exception handling for web.

comment:4 Changed 12 years ago by jburel

  • Sprint set to 2011-06-02 (13)

comment:5 Changed 12 years ago by jburel <j.burel@…>

(In [964af47aaa003a3fe4e1840f195b6ef0db773641/ome.git] on branch develop) Handle ConcurrencyException instead of MissingPyramidException (see #5549)

comment:6 Changed 12 years ago by jburel

  • Owner changed from jburel to cxallan

Passing it to Chris.

comment:7 Changed 12 years ago by jburel

  • Sprint changed from 2011-06-02 (13) to 2011-06-16 (14)

Moved from sprint 2011-06-02 (13)

comment:8 Changed 12 years ago by atarkowska

  • Owner changed from cxallan to wmoore

comment:9 Changed 12 years ago by jburel

  • Remaining Time changed from 0.15 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

See #5539. 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.75979 sec.)

We're Hiring!