Fix for backward compatibility

eBay · Oct 9, 2024 · c6ab600 · c6ab600
1 parent 4483127
commit c6ab600
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/include/libjungle/jungle_builder.h b/include/libjungle/jungle_builder.h
@@ -47,7 +47,7 @@ class Builder {
         /**
          * DBConfig for the DB instance to be created.
          */
-        const DBConfig* dbConfig;
+        DBConfig dbConfig;
 
         /**
          * Detailed table info.

diff --git a/src/jungle_builder.cc b/src/jungle_builder.cc
@@ -31,7 +31,7 @@ Status Builder::buildFromTableFiles(const BuildParams& params) {
     TableMgrOptions t_mgr_opt;
     t_mgr_opt.path = params.path;
     t_mgr_opt.fOps = f_ops.get();
-    t_mgr_opt.dbConfig = params.dbConfig;
+    t_mgr_opt.dbConfig = &params.dbConfig;
 
     MutableTableMgr t_mgr(nullptr);
     t_mgr.setOpt(t_mgr_opt);
@@ -81,7 +81,7 @@ Status Builder::buildFromTableFiles(const BuildParams& params) {
     }
 
     // Create and add empty L0 tables.
-    for (size_t ii = 0; ii < params.dbConfig->numL0Partitions; ++ii) {
+    for (size_t ii = 0; ii < params.dbConfig.numL0Partitions; ++ii) {
         TableFile* t_file = new TableFile(&t_mgr);
         uint64_t table_number = ++max_table_num;
         std::string t_filename =
@@ -102,7 +102,7 @@ Status Builder::buildFromTableFiles(const BuildParams& params) {
     LogMgrOptions l_opt;
     l_opt.path = params.path;
     l_opt.fOps = f_ops.get();
-    l_opt.dbConfig = params.dbConfig;
+    l_opt.dbConfig = &params.dbConfig;
     l_opt.startSeqnum = max_seqnum + 1;
     l_mgr.init(l_opt);
     l_mgr.sync(false);
@@ -121,7 +121,7 @@ Status Builder::init(const std::string& path,
     dbConfig = db_config;
     fOps = new FileOpsPosix();
     buildParams.path = path;
-    buildParams.dbConfig = &dbConfig;
+    buildParams.dbConfig = dbConfig;
 
     if (!fOps->exist(dstPath)) {
         // Create the directory.

diff --git a/tests/jungle/builder_test.cc b/tests/jungle/builder_test.cc
@@ -98,7 +98,7 @@ int build_from_table_files_test() {
     // Build from table files.
     jungle::builder::Builder::BuildParams params;
     params.path = path;
-    params.dbConfig = &d_conf;
+    params.dbConfig = d_conf;
     for (size_t ii = 0; ii < 4; ++ii) {
         jungle::builder::Builder::BuildParams::TableData td;
         td.tableNumber = ii;
@@ -200,7 +200,6 @@ int build_an_empty_db_test() {
     // Build from table files.
     jungle::builder::Builder::BuildParams params;
     params.path = path;
-    params.dbConfig = &d_conf;
     CHK_Z( jungle::builder::Builder::buildFromTableFiles(params) );
 
     // Open it via Jungle API.