Skip to content

Commit

Permalink
Generalizer: Use new append flag instead of delete_existing flag
Browse files Browse the repository at this point in the history
  • Loading branch information
joto committed Apr 12, 2024
1 parent f196471 commit 3f8fff5
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 12 deletions.
5 changes: 2 additions & 3 deletions src/gen/gen-rivers.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,7 @@ gen_rivers_t::gen_rivers_t(pg_conn_t *connection, bool append, params_t *params)
m_timer_prep(add_timer("prep")), m_timer_get(add_timer("get")),
m_timer_sort(add_timer("sort")), m_timer_net(add_timer("net")),
m_timer_remove(add_timer("remove")), m_timer_width(add_timer("width")),
m_timer_write(add_timer("write")),
m_delete_existing(params->has("delete_existing"))
m_timer_write(add_timer("write"))
{
check_src_dest_table_params_exist();

Expand Down Expand Up @@ -327,7 +326,7 @@ SELECT "{id_column}", "{width_column}", "{name_column}", "{geom_column}"
}
timer(m_timer_width).stop();

if (m_delete_existing) {
if (append_mode()) {
dbexec("TRUNCATE {dest}");
}

Expand Down
1 change: 0 additions & 1 deletion src/gen/gen-rivers.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ class gen_rivers_t : public gen_base_t

std::size_t m_num_waterways = 0;
std::size_t m_num_points = 0;
bool m_delete_existing;
};

#endif // OSM2PGSQL_GEN_RIVERS_HPP
5 changes: 2 additions & 3 deletions src/gen/gen-tile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,7 @@ gen_tile_t::gen_tile_t(pg_conn_t *connection, bool append, params_t *params)
{
m_with_group_by = !get_params().get_identifier("group_by_column").empty();

if (get_params().get_bool("delete_existing")) {
m_delete_existing = true;
if (append_mode()) {
dbexec("PREPARE del_geoms (int, int) AS"
" DELETE FROM {dest} WHERE x=$1 AND y=$2");
}
Expand Down Expand Up @@ -55,7 +54,7 @@ uint32_t gen_tile_t::parse_zoom()

void gen_tile_t::delete_existing(tile_t const &tile)
{
if (!m_delete_existing) {
if (!append_mode()) {
return;
}

Expand Down
1 change: 0 additions & 1 deletion src/gen/gen-tile.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ class gen_tile_t : public gen_base_t
private:
std::size_t m_timer_delete;
uint32_t m_zoom;
bool m_delete_existing = false;
bool m_with_group_by = false;
};

Expand Down
4 changes: 0 additions & 4 deletions src/gen/osm2pgsql-gen.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -280,10 +280,6 @@ class genproc_t
params.set("schema", m_dbschema);
}

if (m_append) {
params.set("delete_existing", true);
}

write_to_debug_log(params, "Params (config):");

log_debug("Connecting to database...");
Expand Down

0 comments on commit 3f8fff5

Please sign in to comment.