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

Make error #284

Closed
ankgup87 opened this issue Sep 13, 2014 · 6 comments
Closed

Make error #284

ankgup87 opened this issue Sep 13, 2014 · 6 comments

Comments

@ankgup87
Copy link
Contributor

Hi,

Seeing this error during make:

In file included from db/write_controller.cc:6:0:
./db/write_controller.h:65:3: error: looser throw specifier for 'virtual rocksdb::StopWriteToken::~StopWriteToken()'
./db/write_controller.h:50:11: error: overriding 'virtual rocksdb::WriteControllerToken::~WriteControllerToken() noexcept (true)'
./db/write_controller.h:72:3: error: looser throw specifier for 'virtual rocksdb::DelayWriteToken::~DelayWriteToken()'
./db/write_controller.h:50:11: error: overriding 'virtual rocksdb::WriteControllerToken::~WriteControllerToken() noexcept (true)'

Verified it on Mac OSX and Linux. Can someone please take a look if some recent commit has broken the build?

@ankgup87 ankgup87 added bug Confirmed RocksDB bugs and removed bug Confirmed RocksDB bugs labels Sep 13, 2014
@igorcanadi igorcanadi reopened this Sep 13, 2014
@igorcanadi
Copy link
Collaborator

I can't reproduce, but hopefully got it fixed with 04ce1b2. Can you please confirm?

@ankgup87
Copy link
Contributor Author

Thanks! Commit fixed the issue but now got one more build error:

Linux:
util/bloom.cc: In member function ‘virtual rocksdb::Slice rocksdb::{anonymous}::FullFilterBitsBuilder::Finish(std::unique_ptr<const char []>_)’:
util/bloom.cc:70:20: error: use of deleted function ‘void std::unique_ptr<_Tp [], _Dp>::reset(_Up) [with Up = char; _Tp = const char; _Dp = std::default_delete<const char []>]’
In file included from /opt/rh/devtoolset-1.1/root/usr/lib/gcc/x86_64-redhat-linux/4.7.2/../../../../include/c++/4.7.2/memory:86:0,
from ./include/rocksdb/filter_policy.h:24,
from util/bloom.cc:10:

Mac:
util/bloom.cc: In member function 'virtual rocksdb::Slice rocksdb::{anonymous}::FullFilterBitsBuilder::Finish(std::unique_ptr<const char []>_)':
util/bloom.cc:70:20: error: use of deleted function 'void std::unique_ptr<_Tp [], _Dp>::reset(_Up) [with Up = char; _Tp = const char; _Dp = std::default_delete<const char []>]'
In file included from /opt/local/include/gcc47/c++/memory:86:0,
from ./include/rocksdb/filter_policy.h:24,
from util/bloom.cc:10:

@igorcanadi
Copy link
Collaborator

Hmm. Which compiler do you use? I don't understand why my compiler doesn't catch this.

@igorcanadi
Copy link
Collaborator

@igorcanadi
Copy link
Collaborator

@ankgup87 does it work now?

@ankgup87
Copy link
Contributor Author

Apologies for replaying late. It works now. Thanks a ton! 👍

igorcanadi added a commit that referenced this issue Mar 26, 2015
fix compilation error (same as fix #284)
BusyJay pushed a commit to BusyJay/rocksdb that referenced this issue Jul 25, 2022
* add multibatch write into memtable (facebook#131)

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>

Support write multiple WriteBatch into RocksDB together. These WriteBatch will be assigned sequence number in order and pushed into queue. If a thread is waiting for some state, it could steal some job from work queue.

Signed-off-by: tabokie <xy.tao@outlook.com>

* Improve Multi Batch Write (facebook#154)

* perform like normal pipelined write

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>

* pass enable_multi_thread_write to BuildDBOptions (facebook#170)

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>

* Fix life time of `memtable_write_group` (facebook#171)

* fix life time of memtable_write_group

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>

* Commit pipeline when write a WriteBatch for linearizability (facebook#267)

* support commit pipeline

Signed-off-by: Little-Wallace <bupt2013211450@gmail.com>
Signed-off-by: tabokie <xy.tao@outlook.com>

* format

Signed-off-by: tabokie <xy.tao@outlook.com>

* remove useless code

Signed-off-by: tabokie <xy.tao@outlook.com>

Co-authored-by: Wallace <bupt2013211450@gmail.com>
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