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

scons error: 'virtual int mongo::RocksEngine::flushAllFiles(mongo::OperationContext*, bool)' marked 'override', but does not override #83

Open
CNUser opened this issue Jun 28, 2017 · 1 comment

Comments

@CNUser
Copy link

CNUser commented Jun 28, 2017

I make all steps, but as last I fail when I type 'scons' and the error as follows:
src/mongo/db/modules/rocks/src/rocks_engine.h:114:21: error:

'virtual int mongo::RocksEngine::flushAllFiles(mongo::OperationContext*, bool)' marked 'override', but does not override
virtual int flushAllFiles(OperationContext* opCtx, bool sync) override;
^
In file included from src/mongo/db/modules/rocks/src/rocks_engine.h:47:0,
from src/mongo/db/modules/rocks/src/rocks_init.cpp:39:
src/mongo/db/storage/kv/kv_engine.h:94:17: error: 'virtual int mongo::KVEngine::flushAllFiles(bool)' was hidden [-Werror=overloaded-virtual]
virtual int flushAllFiles(bool sync) {
^
In file included from src/mongo/db/modules/rocks/src/rocks_init.cpp:39:0:
src/mongo/db/modules/rocks/src/rocks_engine.h:114:21: error: by 'virtual int mongo::RocksEngine::flushAllFiles(mongo::OperationContext*, bool)' [-Werror=overloaded-virtual]
virtual int flushAllFiles(OperationContext* opCtx, bool sync) override;

I checkout mongo-3.4.2. I do know that mongo source is not match mongo-rocks. But I do not know how to solve it.

@CNUser CNUser changed the title scons error:error, 'virtual int mongo::RocksEngine::flushAllFiles(mongo::OperationContext*, bool)' marked 'override', but does not override scons error: 'virtual int mongo::RocksEngine::flushAllFiles(mongo::OperationContext*, bool)' marked 'override', but does not override Jun 28, 2017
@igorcanadi
Copy link
Contributor

Hey @CNUser , can you try checking out v3.4 branch https://github.com/mongodb-partners/mongo-rocks/tree/v3.4?

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

2 participants