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

Opened 11 years ago

Closed 11 years ago

Bug: NullPointerException when closing table file

Reported by: spli Owned by: mtbcarroll
Priority: minor Milestone: 5.0.0-beta1
Component: OmeroFs Version: n.a.
Keywords: fs Cc: fs@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: FS Demo 3

Description

The following results in java.lang.NullPointerException at ome.services.RawFileBean.write(RawFileBean.java:402) appearing in the Tables and Blitz logs on Gretzky build 192 (so without PR907(rebased 806) which also changes the table.close() logic).

t=res.newTable(0,'/test.h5')
t.initialize([omero.grid.LongColumn('lc')])
t.close()

Relevant portions of Tables-0.log and Blitz-0.log attached.

This doesn't occur with just

t=res.newTable(0,'/test.h5')
t.close()

Attachments (2)

Blitz.log (20.1 KB) - added by spli 11 years ago.
Tables.log (7.1 KB) - added by spli 11 years ago.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by spli

Changed 11 years ago by spli

comment:1 Changed 11 years ago by jmoore

  • Keywords fs added
  • Milestone changed from Unscheduled to OMERO-5
  • Sprint set to FS Demo 3

comment:2 Changed 11 years ago by jamoore

  • Remaining Time set to 0.25

This code has changed a little on develop, but this is a result of the optimization for when all bytes are written in order. It should be straight-forward to fix the NPE. There's a a bit of question why this code path wasn't "properly" writing the bytes. Might look into that as well.

comment:3 Changed 11 years ago by jamoore

  • Owner changed from jamoore to mtbcarroll

The NPE on save() was taken care of via c2825e455b32ec (PR 905), but the NPE on write() still exists without providing any benefit. Any changes would conflict with PR 995, so I've asked Mark to remove that block there.

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

We're Hiring!