Task #5549 (closed)
Opened 8 years ago
Closed 8 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)
Change History (10)
Changed 8 years ago by jmoore
comment:1 Changed 8 years ago by jburel
comment:2 Changed 8 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 8 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 8 years ago by jburel
- Sprint set to 2011-06-02 (13)
comment:5 Changed 8 years ago by jburel <j.burel@…>
(In [964af47aaa003a3fe4e1840f195b6ef0db773641/ome.git] on branch develop) Handle ConcurrencyException instead of MissingPyramidException (see #5549)
comment:7 Changed 8 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 8 years ago by atarkowska
- Owner changed from cxallan to wmoore
comment:9 Changed 8 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.
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