Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16033 )
Change subject: tablet: fix linker error ...................................................................... tablet: fix linker error Following 97d775ae883181645dfcbbf1c4c18affc130ea78, building on Centos 6 in release mode nets me: lib/../src/kudu/tablet/CMakeFiles/tablet_test_util.dir/tablet_replica-test-base.cc.o: In function `kudu::tablet::TabletReplicaTestBase::RestartReplica()': /data/8/awong/Repositories/kudu/build/release/../../src/kudu/tablet/tablet_replica-test-base.cc:167: undefined reference to `kudu::tablet::BootstrapTablet(scoped_refptr<kudu::tablet::TabletMetadata>, kudu::consensus::RaftConfigPB, kudu: :clock::Clock*, std::shared_ptr<kudu::MemTracker>, scoped_refptr<kudu::rpc::ResultTracker>, kudu::MetricRegistry*, kudu::FileCache*, scoped_refptr<kudu::tablet::TabletReplica>, scoped_refptr<kudu::log::LogAnchorRegistry>, std::shared_pt r<kudu::tablet::Tablet>*, scoped_refptr<kudu::log::Log>*, kudu::consensus::ConsensusBootstrapInfo*)' Seems like the tablet submodule wasn't being linked into tablet-test-util. Also fixed some relevant spacing. Change-Id: I364d83cff7f20868f818270daac4d3529ca84dfc Reviewed-on: http://gerrit.cloudera.org:8080/16033 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tablet/CMakeLists.txt M src/kudu/tablet/tablet_replica-test-base.cc 2 files changed, 20 insertions(+), 19 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/16033 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I364d83cff7f20868f818270daac4d3529ca84dfc Gerrit-Change-Number: 16033 Gerrit-PatchSet: 3 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
