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

Opened 10 years ago

Closed 10 years ago

BUG: QA7907 better MessageSizeMax handling

Reported by: omero-qa Owned by: jamoore
Priority: critical Milestone: 5.0.2
Component: Services Version: 4.4.10
Keywords: n.a. Cc: c.betz@…, mlinkert, dpwrussell
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description (last modified by jamoore)

https://www.openmicroscopy.org/qa2/qa/feedback/7907/

Ice.MemoryLimitException
    reason = "requested 67108869 bytes, maximum allowed is 67108864 bytes (see Ice.MessageSizeMax)"
	at IceInternal.Ex.throwMemoryLimitException(Ex.java:34)
	at IceInternal.BasicStream.expand(BasicStream.java:2515)
	at IceInternal.BasicStream.writeDouble(BasicStream.java:1571)
	at omero.RDouble.__writeImpl(RDouble.java:151)
	at Ice.ObjectImpl.__write(ObjectImpl.java:357)
	at IceInternal.BasicStream$EncapsEncoder10.writePendingObjects(BasicStream.java:4106)
	at IceInternal.BasicStream.writePendingObjects(BasicStream.java:606)
	at omero.api._MetadataStoreDelM.updateObjects(_MetadataStoreDelM.java:266)
	at omero.api.MetadataStorePrxHelper.updateObjects(MetadataStorePrxHelper.java:919)
	at omero.api.MetadataStorePrxHelper.updateObjects(MetadataStorePrxHelper.java:891)
	at ome.formats.OMEROMetadataStoreClient.saveToDB(OMEROMetadataStoreClient.java:2120)
	at ome.formats.importer.ImportLibrary.importMetadata(ImportLibrary.java:368)
	at ome.formats.importer.ImportLibrary.importImage(ImportLibrary.java:550)
	at org.openmicroscopy.shoola.env.data.OMEROGateway.importImage(OMEROGateway.java:6400)
	at org.openmicroscopy.shoola.env.data.OmeroImageServiceImpl.importFile(OmeroImageServiceImpl.java:1350)
	at org.openmicroscopy.shoola.env.data.views.calls.ImagesImporter.importFile(ImagesImporter.java:77)
	at org.openmicroscopy.shoola.env.data.views.calls.ImagesImporter.access$000(ImagesImporter.java:53)
	at org.openmicroscopy.shoola.env.data.views.calls.ImagesImporter$1.doCall(ImagesImporter.java:102)
	at org.openmicroscopy.shoola.env.data.views.BatchCall.doStep(BatchCall.java:144)
	at org.openmicroscopy.shoola.util.concur.tasks.CompositeTask.doStep(CompositeTask.java:226)
	at org.openmicroscopy.shoola.env.data.views.CompositeBatchCall.doStep(CompositeBatchCall.java:126)
	at org.openmicroscopy.shoola.util.concur.tasks.ExecCommand.exec(ExecCommand.java:165)
	at org.openmicroscopy.shoola.util.concur.tasks.ExecCommand.run(ExecCommand.java:276)
	at org.openmicroscopy.shoola.util.concur.tasks.AsyncProcessor$Runner.run(AsyncProcessor.java:91)
	at java.lang.Thread.run(Thread.java:695)

Change History (2)

comment:1 Changed 10 years ago by jamoore

  • Cc mlinkert dpwrussell added
  • Component changed from from QA to Services
  • Description modified (diff)
  • Milestone changed from Unscheduled to 5.0.2
  • Priority changed from minor to critical

Melissa currently has a patch that should alleviate this problem, or at least reduce the situations in which it arises. This seems to be the only method call which ever throws MemoryLimitException and luckily, says Chris Allan, updateObjects can be called repeatedly, rather than in one go.

comment:2 Changed 10 years ago by jamoore

  • Resolution set to fixed
  • Status changed from new to closed

PR opened with Melissa's commit: https://github.com/openmicroscopy/openmicroscopy/pull/2385

I'll push a few minor changes there and then add testing instructions.

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

We're Hiring!