Yingchun Lai has posted comments on this change. ( http://gerrit.cloudera.org:8080/17745 )
Change subject: add http api, get version info ...................................................................... Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/17745/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/17745/3//COMMIT_MSG@7 PS3, Line 7: add http api, get version info Better to improve the commit message, see https://git-scm.com/book/en/v2/Distributed-Git-Contributing-to-a-Project#_commit_guidelines http://gerrit.cloudera.org:8080/#/c/17745/3/src/kudu/server/default_path_handlers.cc File src/kudu/server/default_path_handlers.cc: http://gerrit.cloudera.org:8080/#/c/17745/3/src/kudu/server/default_path_handlers.cc@444 PS3, Line 444: webserver->RegisterPrerenderedPathHandler("/version", "VersionInfo", VersionInfoHandler, We can get version info by CLI tools, via `kudu tserver/master list`, is it necessary to get it by http API? -- To view, visit http://gerrit.cloudera.org:8080/17745 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic25b08b3d3f9bb74e2a877c733ffcaf45f5ece46 Gerrit-Change-Number: 17745 Gerrit-PatchSet: 3 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Comment-Date: Sun, 01 Aug 2021 03:01:54 +0000 Gerrit-HasComments: Yes
