Skip to content

Commit 28064d4

Browse files
committed
Refactor: Remove the last part of the table_sql struct, the name
1 parent 34387bf commit 28064d4

File tree

2 files changed

+8
-48
lines changed

2 files changed

+8
-48
lines changed

src/middle-pgsql.cpp

+7-42
Original file line numberDiff line numberDiff line change
@@ -116,9 +116,10 @@ std::string build_sql(options_t const &options, std::string const &templ)
116116
} // anonymous namespace
117117

118118
middle_pgsql_t::table_desc::table_desc(options_t const &options,
119-
table_sql const &ts)
119+
std::string_view name)
120120
: m_copy_target(std::make_shared<db_target_descr_t>(
121-
options.middle_dbschema, build_sql(options, ts.name), "id"))
121+
options.middle_dbschema, fmt::format("{}_{}", options.prefix, name),
122+
"id"))
122123
{
123124
}
124125

@@ -1201,42 +1202,6 @@ void middle_pgsql_t::wait()
12011202

12021203
namespace {
12031204

1204-
table_sql sql_for_users(middle_pgsql_options const &store_options)
1205-
{
1206-
table_sql sql{};
1207-
1208-
sql.name = "{prefix}_users";
1209-
1210-
return sql;
1211-
}
1212-
1213-
table_sql sql_for_nodes(middle_pgsql_options const &options)
1214-
{
1215-
table_sql sql{};
1216-
1217-
sql.name = "{prefix}_nodes";
1218-
1219-
return sql;
1220-
}
1221-
1222-
table_sql sql_for_ways()
1223-
{
1224-
table_sql sql{};
1225-
1226-
sql.name = "{prefix}_ways";
1227-
1228-
return sql;
1229-
}
1230-
1231-
table_sql sql_for_relations()
1232-
{
1233-
table_sql sql{};
1234-
1235-
sql.name = "{prefix}_rels";
1236-
1237-
return sql;
1238-
}
1239-
12401205
void init_params(params_t *params, options_t const &options)
12411206
{
12421207
std::string const schema = "\"" + options.middle_dbschema + "\".";
@@ -1304,11 +1269,11 @@ middle_pgsql_t::middle_pgsql_t(std::shared_ptr<thread_pool_t> thread_pool,
13041269

13051270
init_params(&m_params, *options);
13061271

1307-
m_tables.nodes() = table_desc{*options, sql_for_nodes(m_store_options)};
1308-
m_tables.ways() = table_desc{*options, sql_for_ways()};
1309-
m_tables.relations() = table_desc{*options, sql_for_relations()};
1272+
m_tables.nodes() = table_desc{*options, "nodes"};
1273+
m_tables.ways() = table_desc{*options, "ways"};
1274+
m_tables.relations() = table_desc{*options, "rels"};
13101275

1311-
m_users_table = table_desc{*options, sql_for_users(m_store_options)};
1276+
m_users_table = table_desc{*options, "users"};
13121277
}
13131278

13141279
void middle_pgsql_t::set_requirements(

src/middle-pgsql.hpp

+1-6
Original file line numberDiff line numberDiff line change
@@ -86,11 +86,6 @@ class middle_query_pgsql_t : public middle_query_t
8686
middle_pgsql_options m_store_options;
8787
};
8888

89-
struct table_sql
90-
{
91-
std::string name;
92-
};
93-
9489
struct middle_pgsql_t : public middle_t
9590
{
9691
middle_pgsql_t(std::shared_ptr<thread_pool_t> thread_pool,
@@ -119,7 +114,7 @@ struct middle_pgsql_t : public middle_t
119114
{
120115
public:
121116
table_desc() = default;
122-
table_desc(options_t const &options, table_sql const &ts);
117+
table_desc(options_t const &options, std::string_view name);
123118

124119
std::string const &schema() const noexcept
125120
{

0 commit comments

Comments
 (0)