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

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

Bug: LifReader static initialization uses ~4MB

Reported by: jamoore Owned by: mlinkert
Priority: major Milestone: Unscheduled
Component: Performance Version: n.a.
Keywords: n.a. Cc: cxallan
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: n.a.

Description

While profiling a standalone Hibernate indexer (no data access whatsoever), I noticed that the 19th item of the top 20 largest object graphs was the LifReader itself with 3.549 MB of static fields. (The 20th was the SHORT_LUT field itself at 3.540 MB).

This initialization should also certainly be delayed. Optimal, would be if it could be freed, but this will likely require the use of ehcache or similar.

Change History (4)

comment:1 Changed 13 years ago by Melissa Linkert <melissa@…>

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

(In [00500beba8b418a6bf127618ea0e341d0ead5e90/bioformats.git]) Initialize LUTs as needed.

This prevents several MB of memory being (possibly needlessly) used.
Closes #6745.

comment:2 Changed 13 years ago by Melissa Linkert <melissa@…>

(In [00500beba8b418a6bf127618ea0e341d0ead5e90/bioformats.git]) Initialize LUTs as needed.

This prevents several MB of memory being (possibly needlessly) used.
Closes #6745.

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

(In [415fb29f150bb7c87acd410348c4f468fc81fb48/bioformats.git]) Initialize LUTs as needed.

This prevents several MB of memory being (possibly needlessly) used.
Closes #6745.
(cherry picked from commit 00500beba8b418a6bf127618ea0e341d0ead5e90)

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

(In [415fb29f150bb7c87acd410348c4f468fc81fb48/bioformats.git]) Initialize LUTs as needed.

This prevents several MB of memory being (possibly needlessly) used.
Closes #6745.
(cherry picked from commit 00500beba8b418a6bf127618ea0e341d0ead5e90)

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

We're Hiring!