Modify

Opened 13 days ago

#15626 new defect

Objects should not be downloaded via Overpass API when reverting changesets

Reported by: mmd Owned by: Upliner
Priority: normal Milestone:
Component: Plugin reverter Version:
Keywords: Cc:

Description

I have activated the setting "Use Overpass API to download objects", and try to revert changeset 54295156 (as an example). In this changeset, node 5245392367 is deleted (https://osmlab.github.io/osm-deep-history/#/node/5245392367).

Unfortunately, the reverted plugin tries to download objects from Overpass API without providing a version number (which isn't possible anyway at this time), nor with a given timestamp for a [date: ....] setting. This results in subsequent error messages, as the node in question cannot be downloaded from Overpass API (it is already deleted).

My recommendation at this time would be to completely ignore the "Download objects via Overpass API" flag when using the reverter, and instead download the respective object version from the Main API. The semantics of downloading objects just don't fit all that well to the Overpass query currently in use.

017-12-03 12:51:01.648 INFORMATION: Defaults for draw.rawgps.layer.audiowpt.pattern differ: ?{ '{name}' | '{desc}' | '{formattedWaypointOffset}' } != 
2017-12-03 12:51:13.138 INFORMATION: GET https://api.openstreetmap.org/api/0.6/changeset/54295156 -> 200
2017-12-03 12:51:13.277 INFORMATION: GET https://api.openstreetmap.org/api/0.6/changeset/54295156/download -> 200
2017-12-03 12:51:13.733 INFORMATION: GET http://lz4.overpass-api.de/api/interpreter?data=%28node%285245392367%29%3B%3E%3B%29%3Bout+meta%3B -> 200 (231 B)
2017-12-03 12:51:13.892 INFORMATION: GET https://api.openstreetmap.org/api/0.6/node/5245392367/1 -> 200 (425 B)
2017-12-03 12:51:14.205 INFORMATION: GET http://lz4.overpass-api.de/api/interpreter?data=%28node%285245392367%29%3B%3E%3B%29%3Bout+meta%3B -> 200 (231 B)
2017-12-03 12:51:14.263 SCHWERWIEGEND: Handled by bug report queue: java.lang.IllegalStateException: Beim Zusammenführen wurde das Ziel-Objekt vom Type Punkt für das Objekt mit der Kennung 5.245.392.367 nicht gefunden
java.lang.IllegalStateException: Beim Zusammenführen wurde das Ziel-Objekt vom Type Punkt für das Objekt mit der Kennung 5.245.392.367 nicht gefunden
	at reverter.DataSetCommandMerger.getMergeTarget(DataSetCommandMerger.java:62)
	at reverter.DataSetCommandMerger.mergeNode(DataSetCommandMerger.java:82)
	at reverter.DataSetCommandMerger.merge(DataSetCommandMerger.java:173)
	at reverter.DataSetCommandMerger.<init>(DataSetCommandMerger.java:46)
	at reverter.ChangesetReverter.getCommands(ChangesetReverter.java:346)
	at reverter.RevertChangesetTask.realRun(RevertChangesetTask.java:113)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:95)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:143)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

Attachments (0)

Change History (0)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain Upliner.
as The resolution will be set.
to The owner will be changed from Upliner to the specified user.
The owner will change to mmd
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from Upliner to anonymous.

Add Comment


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

 
Note: See TracTickets for help on using tickets.