blockchain_import: Make LMDB-specific names general

Also update terms to better distinguish blockchain management/engine
from database type.
This commit is contained in:
warptangent 2016-02-14 11:32:54 -08:00
parent 2b5e155816
commit 0fedce00d5
No known key found for this signature in database
GPG key ID: 0E490BEBFBE4E92D
2 changed files with 33 additions and 33 deletions

View file

@ -77,12 +77,12 @@ using namespace cryptonote;
using namespace epee; using namespace epee;
int parse_db_arguments(const std::string& db_arg_str, std::string& db_engine, int& mdb_flags) int parse_db_arguments(const std::string& db_arg_str, std::string& db_type, int& db_flags)
{ {
std::vector<std::string> db_args; std::vector<std::string> db_args;
boost::split(db_args, db_arg_str, boost::is_any_of("#")); boost::split(db_args, db_arg_str, boost::is_any_of("#"));
db_engine = db_args.front(); db_type = db_args.front();
boost::algorithm::trim(db_engine); boost::algorithm::trim(db_type);
if (db_args.size() == 1) if (db_args.size() == 1)
{ {
@ -103,15 +103,15 @@ int parse_db_arguments(const std::string& db_arg_str, std::string& db_engine, in
continue; continue;
LOG_PRINT_L1("LMDB flag: " << it); LOG_PRINT_L1("LMDB flag: " << it);
if (it == "nosync") if (it == "nosync")
mdb_flags |= MDB_NOSYNC; db_flags |= MDB_NOSYNC;
else if (it == "nometasync") else if (it == "nometasync")
mdb_flags |= MDB_NOMETASYNC; db_flags |= MDB_NOMETASYNC;
else if (it == "writemap") else if (it == "writemap")
mdb_flags |= MDB_WRITEMAP; db_flags |= MDB_WRITEMAP;
else if (it == "mapasync") else if (it == "mapasync")
mdb_flags |= MDB_MAPASYNC; db_flags |= MDB_MAPASYNC;
else if (it == "nordahead") else if (it == "nordahead")
mdb_flags |= MDB_NORDAHEAD; db_flags |= MDB_NORDAHEAD;
else else
{ {
std::cerr << "unrecognized database flag: " << it << ENDL; std::cerr << "unrecognized database flag: " << it << ENDL;
@ -131,7 +131,7 @@ int pop_blocks(FakeCore& simple_core, int num_blocks)
if (simple_core.support_batch) if (simple_core.support_batch)
use_batch = true; use_batch = true;
else else
LOG_PRINT_L0("WARNING: batch transactions enabled but unsupported or unnecessary for this database engine - ignoring"); LOG_PRINT_L0("WARNING: batch transactions enabled but unsupported or unnecessary for this database type - ignoring");
} }
if (use_batch) if (use_batch)
@ -176,11 +176,11 @@ template <typename FakeCore>
int import_from_file(FakeCore& simple_core, const std::string& import_file_path, uint64_t block_stop=0) int import_from_file(FakeCore& simple_core, const std::string& import_file_path, uint64_t block_stop=0)
{ {
#if !defined(BLOCKCHAIN_DB) #if !defined(BLOCKCHAIN_DB)
static_assert(std::is_same<fake_core_memory, FakeCore>::value || std::is_same<fake_core_lmdb, FakeCore>::value, static_assert(std::is_same<fake_core_memory, FakeCore>::value || std::is_same<fake_core_db, FakeCore>::value,
"FakeCore constraint error"); "FakeCore constraint error");
#endif #endif
#if !defined(BLOCKCHAIN_DB) || (BLOCKCHAIN_DB == DB_LMDB) #if !defined(BLOCKCHAIN_DB) || (BLOCKCHAIN_DB == DB_LMDB)
if (std::is_same<fake_core_lmdb, FakeCore>::value) if (std::is_same<fake_core_db, FakeCore>::value)
{ {
// Reset stats, in case we're using newly created db, accumulating stats // Reset stats, in case we're using newly created db, accumulating stats
// from addition of genesis block. // from addition of genesis block.
@ -250,7 +250,7 @@ int import_from_file(FakeCore& simple_core, const std::string& import_file_path,
if (simple_core.support_batch) if (simple_core.support_batch)
use_batch = true; use_batch = true;
else else
LOG_PRINT_L0("WARNING: batch transactions enabled but unsupported or unnecessary for this database engine - ignoring"); LOG_PRINT_L0("WARNING: batch transactions enabled but unsupported or unnecessary for this database type - ignoring");
} }
if (use_batch) if (use_batch)
@ -530,9 +530,9 @@ int import_from_file(FakeCore& simple_core, const std::string& import_file_path,
int main(int argc, char* argv[]) int main(int argc, char* argv[])
{ {
#if defined(BLOCKCHAIN_DB) && (BLOCKCHAIN_DB == DB_MEMORY) #if defined(BLOCKCHAIN_DB) && (BLOCKCHAIN_DB == DB_MEMORY)
std::string default_db_engine = "memory"; std::string default_db_type = "memory";
#else #else
std::string default_db_engine = "lmdb"; std::string default_db_type = "lmdb";
#endif #endif
uint32_t log_level = LOG_LEVEL_0; uint32_t log_level = LOG_LEVEL_0;
@ -567,7 +567,7 @@ int main(int argc, char* argv[])
}; };
const command_line::arg_descriptor<std::string> arg_database = { const command_line::arg_descriptor<std::string> arg_database = {
"database", "available: memory, lmdb" "database", "available: memory, lmdb"
, default_db_engine , default_db_type
}; };
const command_line::arg_descriptor<bool> arg_verify = {"verify", const command_line::arg_descriptor<bool> arg_verify = {"verify",
"Verify blocks and transactions during import", true}; "Verify blocks and transactions during import", true};
@ -677,23 +677,23 @@ int main(int argc, char* argv[])
} }
std::string db_engine; std::string db_type;
int mdb_flags = 0; int db_flags = 0;
int res = 0; int res = 0;
res = parse_db_arguments(db_arg_str, db_engine, mdb_flags); res = parse_db_arguments(db_arg_str, db_type, db_flags);
if (res) if (res)
{ {
std::cerr << "Error parsing database argument(s)" << ENDL; std::cerr << "Error parsing database argument(s)" << ENDL;
return 1; return 1;
} }
if (std::find(db_engines.begin(), db_engines.end(), db_engine) == db_engines.end()) if (std::find(db_engines.begin(), db_engines.end(), db_type) == db_engines.end())
{ {
std::cerr << "Invalid database engine: " << db_engine << std::endl; std::cerr << "Invalid database type: " << db_type << std::endl;
return 1; return 1;
} }
LOG_PRINT_L0("database: " << db_engine); LOG_PRINT_L0("database: " << db_type);
LOG_PRINT_L0("verify: " << std::boolalpha << opt_verify << std::noboolalpha); LOG_PRINT_L0("verify: " << std::boolalpha << opt_verify << std::noboolalpha);
if (opt_batch) if (opt_batch)
{ {
@ -722,31 +722,31 @@ int main(int argc, char* argv[])
// for multi_db_runtime: // for multi_db_runtime:
#if !defined(BLOCKCHAIN_DB) #if !defined(BLOCKCHAIN_DB)
if (db_engine == "lmdb") if (db_type == "lmdb")
{ {
fake_core_lmdb simple_core(m_config_folder, opt_testnet, opt_batch, mdb_flags); fake_core_db simple_core(m_config_folder, opt_testnet, opt_batch, db_flags);
import_from_file(simple_core, import_file_path, block_stop); import_from_file(simple_core, import_file_path, block_stop);
} }
else if (db_engine == "memory") else if (db_type == "memory")
{ {
fake_core_memory simple_core(m_config_folder, opt_testnet); fake_core_memory simple_core(m_config_folder, opt_testnet);
import_from_file(simple_core, import_file_path, block_stop); import_from_file(simple_core, import_file_path, block_stop);
} }
else else
{ {
std::cerr << "database engine unrecognized" << ENDL; std::cerr << "database type unrecognized" << ENDL;
return 1; return 1;
} }
// for multi_db_compile: // for multi_db_compile:
#else #else
if (db_engine != default_db_engine) if (db_type != default_db_engine)
{ {
std::cerr << "Invalid database engine for compiled version: " << db_engine << std::endl; std::cerr << "Invalid database type for compiled version: " << db_type << std::endl;
return 1; return 1;
} }
#if BLOCKCHAIN_DB == DB_LMDB #if BLOCKCHAIN_DB == DB_LMDB
fake_core_lmdb simple_core(m_config_folder, opt_testnet, opt_batch, mdb_flags); fake_core_db simple_core(m_config_folder, opt_testnet, opt_batch, db_flags);
#else #else
fake_core_memory simple_core(m_config_folder, opt_testnet); fake_core_memory simple_core(m_config_folder, opt_testnet);
#endif #endif

View file

@ -47,7 +47,7 @@ namespace
#if !defined(BLOCKCHAIN_DB) || BLOCKCHAIN_DB == DB_LMDB #if !defined(BLOCKCHAIN_DB) || BLOCKCHAIN_DB == DB_LMDB
struct fake_core_lmdb struct fake_core_db
{ {
Blockchain m_storage; Blockchain m_storage;
HardFork* m_hardfork = nullptr; HardFork* m_hardfork = nullptr;
@ -57,10 +57,10 @@ struct fake_core_lmdb
// for multi_db_runtime: // for multi_db_runtime:
#if !defined(BLOCKCHAIN_DB) #if !defined(BLOCKCHAIN_DB)
fake_core_lmdb(const boost::filesystem::path &path, const bool use_testnet=false, const bool do_batch=true, const int mdb_flags=0) : m_pool(&m_storage), m_storage(m_pool) fake_core_db(const boost::filesystem::path &path, const bool use_testnet=false, const bool do_batch=true, const int db_flags=0) : m_pool(&m_storage), m_storage(m_pool)
// for multi_db_compile: // for multi_db_compile:
#else #else
fake_core_lmdb(const boost::filesystem::path &path, const bool use_testnet=false, const bool do_batch=true, const int mdb_flags=0) : m_pool(m_storage), m_storage(m_pool) fake_core_db(const boost::filesystem::path &path, const bool use_testnet=false, const bool do_batch=true, const int db_flags=0) : m_pool(m_storage), m_storage(m_pool)
#endif #endif
{ {
m_pool.init(path.string()); m_pool.init(path.string());
@ -76,7 +76,7 @@ struct fake_core_lmdb
const std::string filename = folder.string(); const std::string filename = folder.string();
try try
{ {
db->open(filename, mdb_flags); db->open(filename, db_flags);
} }
catch (const std::exception& e) catch (const std::exception& e)
{ {
@ -96,7 +96,7 @@ struct fake_core_lmdb
support_batch = true; support_batch = true;
support_add_block = true; support_add_block = true;
} }
~fake_core_lmdb() ~fake_core_db()
{ {
m_storage.get_db().check_hard_fork_info(); m_storage.get_db().check_hard_fork_info();
m_storage.deinit(); m_storage.deinit();