#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)
Change History (8)
Changed 8 years ago by
Attachment: | patch-OsmReaderPerformanceTest.patch added |
---|
comment:1 Changed 8 years ago by
Keywords: | performance added |
---|---|
Milestone: | → 16.02 |
comment:2 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:3 Changed 8 years ago by
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 8 years ago by
I didn't notice. I'll see that I use something else. It simply converts a file to a byte[].
Changed 8 years ago by
Attachment: | patch-remove-ioutils.patch added |
---|
comment:5 Changed 8 years ago by
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.
In 9457/josm: