metadata-extractor icon indicating copy to clipboard operation
metadata-extractor copied to clipboard

Port RandomAccessStream to Java

Open kwhopper opened this issue 5 years ago • 3 comments

@drewnoakes If you decide to go with some version of drewnoakes/metadata-extractor-dotnet#130 I would be happy to attempt a port to Java. My Java skills aren't perfect though.

kwhopper avatar Aug 04 '18 15:08 kwhopper

A reader implementation that doesn't have to read everything into memory would be a great improvement in my opinion :+1: Some images are really slow to parse.

Nadahar avatar Aug 04 '18 15:08 Nadahar

@Nadahar Could you point out some of the images that are slow? I could run some performance tests on the .NET PR vs .NET vs Java and see if RandomAccessStream helps speed things up. That might suggest improvements also.

I don't claim that RAS is perfect or will solve all the performance problems, but I do think that development in the library is difficult because of how fragmented data reading has become. Consolidating the reading layer feels like a necessity. If nothing else RAS puts it all in one place and tuning can be done there independent of the format readers.

kwhopper avatar Aug 04 '18 16:08 kwhopper

@kwhopper I think that any large JPEG would do.

Nadahar avatar Aug 04 '18 18:08 Nadahar