Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/17556 )
Change subject: [rest] add rest to master servers ...................................................................... Patch Set 28: (3 comments) http://gerrit.cloudera.org:8080/#/c/17556/28//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17556/28//COMMIT_MSG@11 PS28, Line 11: nit: extra space http://gerrit.cloudera.org:8080/#/c/17556/28//COMMIT_MSG@14 PS28, Line 14: flags can you add the default port number here? http://gerrit.cloudera.org:8080/#/c/17556/28/src/kudu/master/master.cc File src/kudu/master/master.cc: http://gerrit.cloudera.org:8080/#/c/17556/28/src/kudu/master/master.cc@102 PS28, Line 102: DEFINE_string(rest_server_bind_address,"0.0.0.0:8061","Address where the rest server should start up"); nit: line too long -- 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: comment Gerrit-Change-Id: I0ae85888686dbe3a532f1447aafefb4bc6125ee3 Gerrit-Change-Number: 17556 Gerrit-PatchSet: 28 Gerrit-Owner: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 19 Jul 2021 08:01:06 +0000 Gerrit-HasComments: Yes
