Adar Dembo has posted comments on this change. Change subject: KUDU-1534 : Added software_version to ListMasters RPC ......................................................................
Patch Set 6: (2 comments) I left you some more comments in the JIRA regarding the testing. http://gerrit.cloudera.org:8080/#/c/4099/6/src/kudu/integration-tests/create-table-itest.cc File src/kudu/integration-tests/create-table-itest.cc: PS6, Line 124: const int kNumServers = 3; : const int kNumTablets = 3; Why did you change these values? http://gerrit.cloudera.org:8080/#/c/4099/6/src/kudu/master/master-path-handlers.cc File src/kudu/master/master-path-handlers.cc: PS6, Line 513: std::string Missed one. -- To view, visit http://gerrit.cloudera.org:8080/4099 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I750eeb78c989daaa6fd5d9793b1218ae5993653c Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes