Modify

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#12356 closed enhancement (fixed)

[Patch] OsmReaderPerformanceTest

Reported by: michael2402 Owned by: team
Priority: normal Milestone: 16.02
Component: Core Version:
Keywords: performance Cc:

Description

Tests the performance when parsing an OSM file.

To get rid of disk effects, the file is first read into memeory and parsed afterwards.

Attachments (2)

patch-OsmReaderPerformanceTest.patch (3.1 KB) - added by michael2402 5 years ago.
patch-remove-ioutils.patch (1.7 KB) - added by michael2402 5 years ago.

Download all attachments as: .zip

Change History (8)

Changed 5 years ago by michael2402

comment:1 Changed 5 years ago by Don-vip

Keywords: performance added
Milestone: 16.02

comment:2 Changed 5 years ago by Don-vip

Resolution: fixed
Status: newclosed

In 9457/josm:

fix #12356 - OsmReaderPerformanceTest (patch by michael2402, modified)

comment:3 Changed 5 years ago by Don-vip

This causes a warning:

IOUtils is internal proprietary API and may be removed in a future release

Didn't try it with Java 9 yet but it will likely be a problem. Can we use something else?

comment:4 Changed 5 years ago by michael2402

I didn't notice. I'll see that I use something else. It simply converts a file to a byte[].

Changed 5 years ago by michael2402

Attachment: patch-remove-ioutils.patch added

comment:5 Changed 5 years ago by michael2402

I added a patch that uses plain Java IO. Apache Commons has some UIUtils that can replace this with fewer lines but it does not seem to be in the current dependencies of JOSM.

comment:6 Changed 5 years ago by Don-vip

In 9491/josm:

see #12356 - remove use of internal proprietary API (patch by michael2402)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.