Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mismatch between MongoDB and RocksDB #49

Open
sand-stone opened this issue Dec 12, 2016 · 0 comments
Open

mismatch between MongoDB and RocksDB #49

sand-stone opened this issue Dec 12, 2016 · 0 comments

Comments

@sand-stone
Copy link

I just tried to build from the two latest GitHub source trees. The build instructions are a little out of date, though worked mostly.

Here are some "hacks" to make it work out of the box.

=====
diff --git a/SConscript b/SConscript
index 0cc053c..578a7f0 100644
--- a/SConscript
+++ b/SConscript
@@ -40,6 +40,8 @@ env.Library(
],
SYSLIBDEPS=["rocksdb",
"z",

  •            "snappy",
    
  •            "lz4",
               "bz2"] #z and bz2 are dependencies for rocks
              + dynamic_syslibdeps
    

    )
    diff --git a/src/rocks_engine.cpp b/src/rocks_engine.cpp
    index 2bfbac1..f1bbff9 100644
    --- a/src/rocks_engine.cpp
    +++ b/src/rocks_engine.cpp
    @@ -270,7 +270,7 @@ namespace mongo {
    uint32_t identPrefix = static_cast<uint32_t>(element.numberInt());

               _identMap[StringData(ident.data(), ident.size())] =
    
  •                std::move(identConfig.getOwned());
    
  •                identConfig.getOwned();
    
               _maxPrefix = std::max(_maxPrefix, identPrefix);
           }
    

@@ -566,7 +566,7 @@ namespace mongo {
prefix = ++_maxPrefix;
configBuilder->append("prefix", static_cast<int32_t>(prefix));

  •        config = std::move(configBuilder->obj());
    
  •        config = configBuilder->obj();
           _identMap[ident] = config.copy();
       }
    
igorsol added a commit to igorsol/mongo-rocks that referenced this issue May 12, 2017
PSMDB-138 Use global unique snapshot ID for RecoveryUnit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant