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

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

Bug; Directory scanning and failure

Reported by: jburel Owned by: mlinkert
Priority: major Milestone: OMERO-4.4.7
Component: Import Version: n.a.
Keywords: n.a. Cc: jamoore
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description

  • Select a directory containing a file that cannot be imported (Out of Memory Exception) and other files that can be imported
  • The ImportCandidates method throws an exception so the valid files are not returned.
  • Ideally all files should be returned the ones that can be imported and the one that cannot be imported +Error

Change History (4)

comment:1 Changed 11 years ago by mlinkert

  • Cc jmoore added

This is not a problem in Bio-Formats, but rather in ImportCandidates, right? ImportCandidates already calls setId in a try/catch block, so I guess that it is just a matter of doing something different when Exception is caught.

comment:2 Changed 11 years ago by jburel

wrong component selected when creating the ticket.

comment:3 Changed 11 years ago by mlinkert

  • Component changed from Bio-Formats to Import
  • Resolution set to fixed
  • Status changed from new to closed

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

  • Remaining Time set to 0

(In [52ebbf1965c77673df2dbcb66b93224441cafcff/ome.git] on branch develop) ImportCandidates?: catch Throwable instead of Exception

This allows us to continue if, for instance, an OOM is thrown during
setId.

Fixes #9872.

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

We're Hiring!