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

Opened 9 years ago

Closed 9 years ago

Add test for leaking file handles

Reported by: mlinkert Owned by: mlinkert
Priority: minor Milestone: OMERO-Beta4.3
Component: Bio-Formats Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2011-04-21 (10)

Description

File handles that are left open are known to cause a problem on Windows, so we should somehow find a way to test that there are no open file handles after close() has been called on a reader.

Change History (6)

comment:1 Changed 9 years ago by jmoore

imported from bio-formats:#532

comment:2 Changed 9 years ago by mlinkert

  • Remaining Time set to 0.5
  • Sprint set to 2011-04-21 (10)

comment:3 Changed 9 years ago by Melissa Linkert <melissa@…>

  • Remaining Time changed from 0.5 to 0
  • Resolution set to fixed
  • Status changed from new to closed

(In [ba30664b23781cf0e45ca855de662354c9681a70/bioformats.git]) Add test for leaking file handles. Closes #4139.

It does not work on Windows, but is better than nothing.

comment:4 Changed 9 years ago by Melissa Linkert <melissa@…>

(In [ba30664b23781cf0e45ca855de662354c9681a70/bioformats.git]) Add test for leaking file handles. Closes #4139.

It does not work on Windows, but is better than nothing.

comment:5 Changed 9 years ago by jmoore

Melissa, on our test systems I typically have System Internals installed if you'd like to add a test which uses "handle": http://stackoverflow.com/questions/15708/lsof-equivalent-for-windows

comment:6 Changed 9 years ago by mlinkert

Thanks for the suggestion Josh - I've just filed #5258 to add a test that uses 'handle'.

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

We're Hiring!