You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2022/04/20 23:49:03 UTC

[kudu] branch master updated: [unit test] Fix a unit test problem from code review.

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 42fbe6b2e [unit test] Fix a unit test problem from code review.
42fbe6b2e is described below

commit 42fbe6b2e0085d934068e1687de1d3c936e89ed1
Author: shenxingwuying <sh...@gmail.com>
AuthorDate: Wed Apr 20 16:14:44 2022 +0800

    [unit test] Fix a unit test problem from code review.
    
    Internal MiniTabletServer's Restart() function would core dump,
    because server_ may be not nullptr, should shutdown and then start.
    
    Change-Id: Ib6a810799cdd9adec24944c4bea959103c775966
    Reviewed-on: http://gerrit.cloudera.org:8080/18427
    Tested-by: Alexey Serbin <al...@apache.org>
    Reviewed-by: Alexey Serbin <al...@apache.org>
---
 src/kudu/tserver/mini_tablet_server.cc | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/kudu/tserver/mini_tablet_server.cc b/src/kudu/tserver/mini_tablet_server.cc
index 43c76d900..40acd362a 100644
--- a/src/kudu/tserver/mini_tablet_server.cc
+++ b/src/kudu/tserver/mini_tablet_server.cc
@@ -126,6 +126,7 @@ void MiniTabletServer::Shutdown() {
 }
 
 Status MiniTabletServer::Restart() {
+  Shutdown();
   return Start();
 }