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 <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
