Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19613 )

Change subject: Exit external_mini_cluster services when test is stopped
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/19613/2/src/kudu/master/master_runner.cc
File src/kudu/master/master_runner.cc:

http://gerrit.cloudera.org:8080/#/c/19613/2/src/kudu/master/master_runner.cc@467
PS2, Line 467:   while (true) {
             :     SleepFor(MonoDelta::FromSeconds(1));
             :     if (FLAGS_exit_if_orphaned && getppid() == 1) return 
Status::OK();
             :   }
> > It's perfectly valid to start kudu master with "kudu master run" command,
Ah, maybe your concern was about running 'kudu master/tserver run' in 
Docker-based scenarios?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7ca88b3b8924dc7dc5c05a0239dfa86db67af255
Gerrit-Change-Number: 19613
Gerrit-PatchSet: 2
Gerrit-Owner: Ádám Bakai <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Ádám Bakai <[email protected]>
Gerrit-Comment-Date: Sun, 16 Apr 2023 16:39:31 +0000
Gerrit-HasComments: Yes

Reply via email to