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 (#25). Change subject: [rest] add rest to master servers ...................................................................... [rest] add rest to master servers As the rest server uses a kudu client for the requests, and kudu client needs to know the addresses of the master servers, the better choice was to embed into the master, instead of having a seperate server . This also has the benefit of keeping the rest endpoint alive in case one of the master servers stops. The rest endpoint is off by default and requires a bind address, which can be provided by flags. A seperate threadpool takes care of running the rest endpoint, not to block the master server. Change-Id: I0ae85888686dbe3a532f1447aafefb4bc6125ee3 --- M src/kudu/master/CMakeLists.txt M src/kudu/master/master.cc M src/kudu/master/master.h 3 files changed, 27 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/17556/25 -- 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: 25 Gerrit-Owner: Khazar Mammadli <mammadli.kha...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120)