Hello Attila Bukor, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17556 to look at the new patch set (#2). Change subject: [rest] add rest to master servers ...................................................................... [rest] add rest to master servers Modify master to have a rest server as a member. Flags rest_server_enabled and rest_bind_address need to be set to start up an endpoint Initialization of the rest endpoints is done in the constructor of master if the flags are set. The master has a threadpool for rest, because rest run() blocks the thread. Server only starts up, if it was initialized during the construction of the master. Shutting down the master, shuts down the threadpool running the rest. master/CMakeLists.txt modify to have the rest target. Change-Id: I0ae85888686dbe3a532f1447aafefb4bc6125ee3 --- M src/kudu/master/CMakeLists.txt M src/kudu/master/master.cc M src/kudu/master/master.h 3 files changed, 19 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/17556/2 -- To view, visit http://gerrit.cloudera.org:8080/17556 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I0ae85888686dbe3a532f1447aafefb4bc6125ee3 Gerrit-Change-Number: 17556 Gerrit-PatchSet: 2 Gerrit-Owner: Khazar Mammadli <mammadli.kha...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120)