From: Sarah Hoffmann Date: Mon, 23 Mar 2015 21:01:52 +0000 (+0100) Subject: Merge remote-tracking branch 'upstream/master' X-Git-Tag: deploy~472 X-Git-Url: https://git.openstreetmap.org./nominatim.git/commitdiff_plain/de978c9987ddbb6595029b23b5dfd2823fc4ea3f?ds=inline;hp=-c Merge remote-tracking branch 'upstream/master' --- de978c9987ddbb6595029b23b5dfd2823fc4ea3f diff --combined sql/functions.sql index c3590c1e,5f52a6a9..96b5eaaf --- a/sql/functions.sql +++ b/sql/functions.sql @@@ -1734,7 -1734,7 +1734,7 @@@ BEGI END IF; -- Name searches can be done for ways as well as relations - IF NEW.osm_type in ('W','R') AND NEW.rank_search < 26 THEN + IF NEW.osm_type in ('W','R') AND NEW.rank_search < 26 AND NEW.rank_address > 0 THEN -- not found one yet? how about doing a name search IF NEW.centroid IS NULL AND (NEW.name->'name') is not null and make_standard_name(NEW.name->'name') != '' THEN @@@ -2118,11 -2118,6 +2118,11 @@@ BEGI --DEBUG: RAISE WARNING '%', existingplacex; END IF; + -- remove operator tag for most places, messes too much with search_name indexes + IF NEW.class not in ('amenity', 'shop') THEN + NEW.name := delete(NEW.name, 'operator'); + END IF; + -- Just block these - lots and pointless IF NEW.class in ('landuse','natural') and NEW.name is null THEN -- if the name tag was removed, older versions might still be lurking in the place table @@@ -2312,12 -2307,6 +2312,12 @@@ END IF; END IF; + + -- refuse to update multiplpoygons with too many objects, too much of a performance hit + IF ST_NumGeometries(NEW.geometry) > 2000 THEN + RAISE WARNING 'Dropping update of % % because of geometry complexity.', NEW.osm_type, NEW.osm_id; + RETURN NULL; + END IF; IF coalesce(existing.name::text, '') != coalesce(NEW.name::text, '') OR coalesce(existing.extratags::text, '') != coalesce(NEW.extratags::text, '')