Project

General

Profile

Actions
Copy link

bug #9610

open

MediaMetadataFileReader (formerly in CdmImageInfo) loads same image file multiple times

Added by Andreas Kohlbecker almost 3 years ago. Updated 2 months ago.

Status:
In Progress
Priority:
Highest
Assignee:
Katja Luther
Category:
cdmlib
Target version:
Release 5.43
Start date:
Due date:
% Done:

40%

Estimated time:
Severity:
normal
Found in Version:
Tags:
performance

Description

For one and the same image file the UriUtils methos getInputStream() and getResourceLength() are being used multiple times:

  • getInputStream(): 4x
  • getResourceLength(): 1x

This will cause in the worst case that the file is loaded 5x from the internet! What as waste of time an resources!!!

Related issues

Related to EDIT - bug #9608: UriUtils to use HTTP HEAD requests where possibleNewKatja Luther

Actions
Related to EDIT - task #9607: read image metadata from MediaInfoService if availableClosedAndreas Kohlbecker

Actions
Related to EDIT - feature request #5016: Use asynchronous loading for imagesNewAndreas Müller

Actions
Actions
Copy link
 #1

Updated by Andreas Kohlbecker almost 3 years ago

  • Related to bug #9608: UriUtils to use HTTP HEAD requests where possible added
Actions
Copy link
 #2

Updated by Andreas Kohlbecker almost 3 years ago

  • Related to task #9607: read image metadata from MediaInfoService if available added
Actions
Copy link
 #4

Updated by Andreas Kohlbecker almost 3 years ago

  • Target version changed from Release 5.25 to Release 5.45
Actions
Copy link
 #5

Updated by Katja Luther about 2 years ago

  • Subject changed from MediaMedadataFileReader (formerly in CdmImageInfo) loads same image file multiple times to MediaMetadataFileReader (formerly in CdmImageInfo) loads same image file multiple times
Actions
Copy link
 #6

Updated by Andreas Müller about 2 years ago

  • Subject changed from MediaMetadataFileReader (formerly in CdmImageInfo) loads same image file multiple times to MediaMedadataFileReader (formerly in CdmImageInfo) loads same image file multiple times
  • Assignee changed from Andreas Kohlbecker to Andreas Müller
  • Target version changed from Release 5.45 to Release 5.44

Check if this is not yet fixed in the meanwhile.

Actions
Copy link
 #7

Updated by Andreas Müller about 2 years ago

  • Subject changed from MediaMedadataFileReader (formerly in CdmImageInfo) loads same image file multiple times to MediaMetadataFileReader (formerly in CdmImageInfo) loads same image file multiple times
Actions
Copy link
 #8

Updated by Andreas Müller about 2 years ago

Actions
Copy link
 #9

Updated by Andreas Müller 2 months ago

  • Status changed from New to Feedback
  • Assignee changed from Andreas Müller to Katja Luther

As far as I remember we have worked on this some time ago. Can you please check if you think that this still happens?

Actions
Copy link
 #10

Updated by Andreas Müller 2 months ago

  • Target version changed from Release 5.44 to Release 5.43
Actions
Copy link
 #11

Updated by Katja Luther 2 months ago

In dataportal getInputStream() is only called twice

But for the editor getInputStream() is called 4 times!!!
The first call is only to test whether the file exists.
Two calls in MediaInfoFileReader (baseInfo() and readMetaData()) and a fourth call to show the image.

Actions
Copy link
 #12

Updated by Katja Luther 2 months ago

  • Status changed from Feedback to In Progress
  • % Done changed from 0 to 40

I adapted the MediaInfoFileReader in a way that the imageInfo and the metadata using the same inputStream.

So currently we have for the dataportal only one call of getInputStream() and in TaxEditor two calls, the second call is caused by the ImageElement which displays the image.

Actions
Copy link

Also available in: Atom PDF