Task #7747 (closed)
Bug: enableKeepAlive hangs python
| Reported by: | jamoore | Owned by: | jamoore |
|---|---|---|---|
| Priority: | minor | Milestone: | OMERO-4.4 |
| Component: | General | Version: | n.a. |
| Keywords: | n.a. | Cc: | |
| Resources: | n.a. | Referenced By: | n.a. |
| References: | n.a. | Remaining Time: | 0.0d |
| Sprint: | 2012-01-17 (6) |
Description
The script from #2973 hangs after createSession throws (with a bad login) since enableKeepAlive created a thread. A thread should only be activated when a session is present. Otherwise, i.e. after closeSession and on a failed createSession, there should be no actual thread, only the keep alive setting should be modified.
Change History (4)
comment:1 Changed 12 years ago by jmoore
- Status changed from new to accepted
comment:2 Changed 12 years ago by jmoore
- Remaining Time changed from 0.1 to 0
- Resolution set to fixed
- Status changed from accepted to closed
comment:3 Changed 12 years ago by jmoore
Forgot to mention: pushed to sprint6-server-backlog
comment:4 Changed 12 years ago by jmoore <josh@…>
(In [c301e9f9460e461d2abda80655d4c3a1d217eea6/ome.git] on branch develop) Refactor enableKeepAlive to not hang (Fix #7747)
Note: See
TracTickets for help on using
tickets.
You may also have a look at Agilo extensions to the ticket.
Refactored startKeepAlive and stopKeepAlive out of enableKeepAlive. createSession and closeSession now use them more appropriately. Added mock tests to show the possibilities.