search

jubatus / jubatus_core

Jubatus algorithm component
GNU Lesser General Public License v2.1
20 stars 29 forks source link

Fix double acquision of a lock in the same thread #342

Closed rimms closed 7 years ago

rimms commented 7 years ago

Refs. https://github.com/jubatus/jubatus_core/pull/312

$ ./waf configure --enable-debug
$ ./waf build --checkall
 .. snip..
/path/to/jubatus_core/build/jubatus/core/classifier/classifier_test
 ..snip..
[ RUN      ] cl/classifier_test/7.delete_label
classifier_test: ../jubatus/util/concurrent/rwmutex.cpp:106: bool jubatus::util::concurrent::rw_mutex::impl::read_lock(): Assertion `holders.count(thread::id())==0' failed.

/path/to/jubatus_core/build/jubatus/core/driver/nearest_neighbor_test
 .. snip ..
[ RUN      ] nearest_neighbor_with_unlearning_test_instance/nearest_neighbor_with_unlearning_test.unlearning/0
nearest_neighbor_test: ../jubatus/util/concurrent/rwmutex.cpp:106: bool jubatus::util::concurrent::rw_mutex::impl::read_lock(): Assertion `holders.count(thread::id())==0' failed.

/path/to/jubatus/jubatus_core/build/jubatus/core/anomaly/light_lof_test
 ..snip ..
[ RUN      ] usual_case/light_lof_test/0.get_all_row_ids
light_lof_test: ../jubatus/util/concurrent/rwmutex.cpp:106: bool jubatus::util::concurrent::rw_mutex::impl::read_lock(): Assertion `holders.count(thread::id())==0' failed.

/path/to/jubatus/jubatus_core/build/jubatus/core/driver/anomaly_test
 .. snip ..
[ RUN      ] light_lof_test.unlearning
anomaly_test: ../jubatus/util/concurrent/rwmutex.cpp:106: bool jubatus::util::concurrent::rw_mutex::impl::read_lock(): Assertion `holders.count(thread::id())==0' failed.
kmaehashi commented 7 years ago

Fixed via #350.