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 (#8). 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 best choice was to embed into the master. 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, 19 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/17556/8 -- 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: 8 Gerrit-Owner: Khazar Mammadli <mammadli.kha...@cloudera.com> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120)