Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18238 )

Change subject: [tserver] avoid copying Schema in SplitKeyRange()
......................................................................


Patch Set 1: Verified+1 Code-Review+2

The test failures seem unrelated, though perhaps this one's related to the 
recent Thread change?

threadpool.cc:306] Check failed: 1 == tokens_.size() (1 vs. 3) Threadpool raft 
destroyed with 3 allocated tokens
    @           0x477930 __tsan::CallUserSignalHandler() at 
/home/jenkins-slave/workspace/kudu-master/2/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_interceptors.cc:1916
    @           0x479e47 rtl_sigaction() at 
/home/jenkins-slave/workspace/kudu-master/2/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_interceptors.cc:2006
    @     0x7f81497f3fb7 gsignal at ??:0
    @     0x7f81497f5921 abort at ??:0
    @           0x480f66 abort at 
/home/jenkins-slave/workspace/kudu-master/2/thirdparty/src/llvm-9.0.0.src/projects/compiler-rt/lib/tsan/rtl/tsan_interceptors.cc:261
    @     0x7f814c979e47 kudu::ThreadPool::~ThreadPool() at ??:0
    @     0x7f8153c3854f std::__1::default_delete<>::operator()() at ??:0
    @     0x7f8153c384be std::__1::unique_ptr<>::reset() at ??:0
    @     0x7f8153bf7cfc std::__1::unique_ptr<>::~unique_ptr() at ??:0
    @     0x7f8153c98b54 kudu::kserver::KuduServer::~KuduServer() at ??:0
    @     0x7f8153c916eb kudu::master::Master::~Master() at ??:0
    @     0x7f8153c9199a kudu::master::Master::~Master() at ??:0
    @     0x7f8153cc8c78 std::__1::default_delete<>::operator()() at ??:0
    @     0x7f8153cc1f9e std::__1::unique_ptr<>::reset() at ??:0
    @     0x7f8153cd8393 kudu::master::MiniMaster::Shutdown() at ??:0
    @           0x4ff669 
kudu::tools::RemoteKsckTest_TestClusterWithLocation_Test::TestBody() at 
/home/jenkins-slave/workspace/kudu-master/2/src/kudu/tools/ksck_remote-test.cc:606
    @     0x7f8154951e1c RUN_ALL_TESTS() at ??:0
    @     0x7f8154950c6a main at ??:0
    @     0x7f81497d6bf7 __libc_start_main at ??:0
    @           0x4433aa _start at ??:?

https://gerrit.cloudera.org/c/18229/

Looking through a bit, I don't think so though. Will dig a bit more.


--
To view, visit http://gerrit.cloudera.org:8080/18238
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I07642a438a4eeee6812828ddd871534bdf985155
Gerrit-Change-Number: 18238
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 17 Feb 2022 01:10:48 +0000
Gerrit-HasComments: No

Reply via email to