Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/16570
Change subject: [test] add ReplicatedAlterTableTest.AlterTableAndDropTablet ...................................................................... [test] add ReplicatedAlterTableTest.AlterTableAndDropTablet This patch adds a new test scenario to verify that a set of DDL operations (e.g., ALTER TABLE adding a new column) followed by dropping a tablet are processed with now issues even in case of slow WAL. In addition, this scenario covers the regression fixed with changelist 81f0dbf99. I manually verified that the test fails (i.e. the process crashes) with the stack trace as below if rolling back changelist 81f0dbf99 (macOS DEBUG build): I1009 00:11:52.782909 263938048 op_tracker.cc:285] Dumping up to 50 currently running ops: Assertion failed: (this->is_initialized()), function operator->, file /Users/aserbin/Projects/kudu/thirdparty/installed/uninstrumented/include/boost/optional/optional.hpp, line 1221. *** Aborted at 1602227512 (unix time) try "date -d @1602227512" if you are using GNU date *** PC: @ 0x7fff79a74b66 __pthread_kill *** SIGABRT (@0x7fff79a74b66) received by PID 67923 (TID 0x70000fbb6000) stack trace: *** @ 0x7fff79c32f5a _sigtramp @ 0x10f17a3f5 google::protobuf::TextFormat::Printer::Print() @ 0x7fff799d01ae abort @ 0x7fff799981ac __assert_rtn @ 0x10c2a4c9d boost::optional<>::operator->() @ 0x10c2a49e5 kudu::tablet::AlterSchemaOpState::ToString() @ 0x10c2a6b0c kudu::tablet::AlterSchemaOp::ToString() @ 0x10c2ab03e kudu::tablet::OpDriver::ToStringUnlocked() @ 0x10c2aaec8 kudu::tablet::OpDriver::ToString() @ 0x10c2bb653 kudu::tablet::OpTracker::WaitForAllToFinish() @ 0x10c2baca2 kudu::tablet::OpTracker::WaitForAllToFinish() @ 0x10c272a5f kudu::tablet::TabletReplica::Stop() @ 0x109d3524c kudu::tserver::TSTabletManager::DeleteTablet() @ 0x109d5d542 kudu::tserver::DeleteTabletRunnable::Run() @ 0x109d5e9f1 kudu::tserver::TSTabletManager::DeleteTabletAsync()::$_14::operator()() @ 0x109d5e9bd _ZNSt3__128__invoke_void_return_wrapperIvE6__callIJRZN4kudu7tserver15TSTabletManager17DeleteTabletAsyncERKNS_12basic_stringIcNS_11char_traitsIcEENS_9allocatorIcEEEENS3_6tablet15TabletDataStateERKN5boost8optionalIxEERKNS_8functionIFvRKNS3_6StatusENS4_24TabletServerErrorPB_CodeEEEEE4$_14EEEvDpOT_ @ 0x109d5e6f9 std::__1::__function::__func<>::operator()() @ 0x10913ef46 std::__1::function<>::operator()() @ 0x10e7c22cd kudu::ThreadPool::DispatchThread() @ 0x10e7de7f8 kudu::ThreadPool::CreateThread()::$_1::operator()() @ 0x10e7de7cd _ZNSt3__128__invoke_void_return_wrapperIvE6__callIJRZN4kudu10ThreadPool12CreateThreadEvE3$_1EEEvDpOT_ @ 0x10e7de6e9 std::__1::__function::__func<>::operator()() @ 0x10913ef46 std::__1::function<>::operator()() @ 0x10e7a3275 kudu::Thread::SuperviseThread() @ 0x7fff79c3c661 _pthread_body @ 0x7fff79c3c50d _pthread_start @ 0x7fff79c3bbf9 thread_start Change-Id: I264e9873399ede60b642bde1a1e9c343dfa44072 --- M src/kudu/integration-tests/alter_table-test.cc 1 file changed, 67 insertions(+), 16 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/70/16570/1 -- To view, visit http://gerrit.cloudera.org:8080/16570 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I264e9873399ede60b642bde1a1e9c343dfa44072 Gerrit-Change-Number: 16570 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>