[Tile-serving] [openstreetmap/osm2pgsql] Optionally wrap polygon geometries in multipolygons (#1327)

[Jochen Topf]

@joto commented on this pull request.



>                             osmium::Relation const &relation,
                            osmium::memory::Buffer const &buffer) const
 {
     assert(builder);
 
-    return builder->get_wkb_multipolygon(relation, buffer, m_multi);
+    auto create_multigeoms = geom::create_multi_t::never;
+    if (target_geom_type == table_column_type::multipolygon) {
+        create_multigeoms = geom::create_multi_t::always;
+    } else if (m_multi) {
+        create_multigeoms = geom::create_multi_t::optional;
+    }

After some more thought, this wasn't the right choice. The `multi` setting should be independent of the target geometry type. The new code implements this that way.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/osm2pgsql/pull/1327#discussion_r527038235
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/tile-serving/attachments/20201119/1e4c14cd/attachment.htm>