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.

Sign up for GitHub

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

Jump to bottom

Windows cumulative patch #3552

Closed
wants to merge 15 commits into from
Closed

Windows cumulative patch #3552

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
f5ca24a
Close info_log logger so DestroyDB can cleanup the directory and let
yuslepukhin Feb 13, 2018
68eae93
Adjust windows logger for the new interface pattern.
yuslepukhin Feb 28, 2018
01d1830
Replace std::thread reference and fix the %z formatting.
yuslepukhin Feb 28, 2018
02d9480
Bring scope status var fixes.
yuslepukhin Feb 17, 2018
aeac697
Make use of GetFileInformationByHandleEx() so we make use of the unique
yuslepukhin Feb 21, 2018
d5bdef6
Replace C-runtime in env implementation.
yuslepukhin Feb 22, 2018
5309831
Implement AreFilesTheSame
yuslepukhin Mar 1, 2018
8e66b3e
Align unbuffered read offsets to sector size.
yuslepukhin Feb 16, 2018
0d13a75
Fix DBTest.SoftLimit. The test is based on a premise that
yuslepukhin Mar 1, 2018
0a564b2
Address windows release unused var warning.
yuslepukhin Mar 2, 2018
8f4a373
Address #endif extra chars error
yuslepukhin Mar 2, 2018
93ae4cf
Adjust for MINGW builds.
yuslepukhin Mar 2, 2018
df06f9f
Address DBTest2.ReadAmpBitmapLiveInCacheAfterDBClose failure
yuslepukhin Mar 2, 2018
7469117
Fix WriteStallListenermember initialization.
yuslepukhin Mar 5, 2018
c0c4b8e
Reset expected_set_ in a signalling thread unless there is a timeout
yuslepukhin Mar 6, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion db/db_test.cc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5404,6 +5404,7 @@ class WriteStallListener : public EventListener {
if (expected_set_ &&
condition_ == expected_) {
cond_.Signal();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yuslepukhin ASAN complains this:

Failed to find test summary. This test binary may have crashed mid-run. Full output follows:
Note: Google Test filter = DBTest.SoftLimit
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from DBTest
[ RUN      ] DBTest.SoftLimit
internal_repo_rocksdb/repo/db/db_test.cc:5391:9: runtime error: load of value 190, which is not a valid value for type 'bool'
    #0 0x681976 in rocksdb::WriteStallListener::OnStallConditionsChanged(rocksdb::WriteStallInfo const&) internal_repo_rocksdb/repo/db/db_test.cc:5391
    #1 0x7f7cf9d91494 in rocksdb::SuperVersionContext::Clean() internal_repo_rocksdb/repo/db/job_context.h:63
    #2 0x7f7cf9d826e2 in rocksdb::JobContext::Clean() internal_repo_rocksdb/repo/db/job_context.h:162
    #3 0x7f7cf9f33d83 in rocksdb::DBImpl::BackgroundCallFlush() internal_repo_rocksdb/repo/db/db_impl_compaction_flush.cc:1404
    #4 0x7f7cf9f30fe5 in rocksdb::DBImpl::BGWorkFlush(void*) internal_repo_rocksdb/repo/db/db_impl_compaction_flush.cc:1250
    #5 0x7f7cfa6af0f1 in rocksdb::ThreadPoolImpl::Schedule(void (*)(void*), void*, void*, void (*)(void*))::$_1::operator()() const internal_repo_rocksdb/repo/util/threadpool_imp.cc:441
    #6 0x7f7cfa6aeefc in std::_Function_handler<void (), rocksdb::ThreadPoolImpl::Schedule(void (*)(void*), void*, void*, void (*)(void*))::$_1>::_M_invoke(std::_Any_data const&) third-party-buck/gcc-5-glibc-2.23/build/libgcc/include/c++/trunk/functional:1871
    #7 0x7f7cfa18325e in std::function<void ()>::operator()() const third-party-buck/gcc-5-glibc-2.23/build/libgcc/include/c++/trunk/functional:2267
    #8 0x7f7cfa6ab557 in rocksdb::ThreadPoolImpl::Impl::BGThread(unsigned long) internal_repo_rocksdb/repo/util/threadpool_imp.cc:240
    #9 0x7f7cfa6ab839 in rocksdb::ThreadPoolImpl::Impl::BGThreadWrapper(void*) internal_repo_rocksdb/repo/util/threadpool_imp.cc:278
    #10 0x7f7cfa6b560e in void* std::_Bind_simple<void* (*(rocksdb::BGThreadMetadata*))(void*)>::_M_invoke<0ul>(std::_Index_tuple<0ul>) third-party-buck/gcc-5-glibc-2.23/build/libgcc/include/c++/trunk/functional:1530
    #11 0x7f7cfa6b5574 in std::_Bind_simple<void* (*(rocksdb::BGThreadMetadata*))(void*)>::operator()() third-party-buck/gcc-5-glibc-2.23/build/libgcc/include/c++/trunk/functional:1520
    #12 0x7f7cfa6b52a8 in std::thread::_Impl<std::_Bind_simple<void* (*(rocksdb::BGThreadMetadata*))(void*)> >::_M_run() third-party-buck/gcc-5-glibc-2.23/build/libgcc/include/c++/trunk/thread:115
    #13 0x7f7cf6e56170 in execute_native_thread_routine /home/engshare/third-party2/libgcc/5.x/src/gcc-5/x86_64-facebook-linux/libstdc++-v3/src/c++11/../../../.././libstdc++-v3/src/c++11/thread.cc:84
    #14 0x7f7cf4cab7a8 in start_thread /home/engshare/third-party2/glibc/2.23/src/glibc-2.23/nptl/pthread_create.c:333
    #15 0x7f7cf44d7a7c in __clone /home/engshare/third-party2/glibc/2.23/src/glibc-2.23/misc/../sysdeps/unix/sysv/linux/x86_64/clone.S:109

I assume it's because expected_set_ has never been initialized?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@siying My bad.

expected_set_ = false;
}
}
bool CheckCondition(WriteStallCondition expected) {
Expand All @@ -5418,7 +5419,6 @@ class WriteStallListener : public EventListener {
expected_set_ = false;
return false;
}
expected_set_ = false;
}
}
return true;
Expand Down