You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Junegunn Choi (Code Review)" <ge...@cloudera.org> on 2017/02/13 05:46:12 UTC
[kudu-CR] Add list of tablet servers to /dump-entities
Junegunn Choi has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5977
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This extends the output of /dump-entities endpoint of master server to
include the list of tablet servers.
Background:
We're evaluating Kudu as an alternative to HBase. We realized that Kudu
does not offer an automatic tablet balancer that evenly distributes
tablets across the nodes a la HBase, so we decided to write an external
tool for that purpopse. In order to do that, we need the followings.
1. API for adding and removing replicas
Kudu does not yet have a set of APIs for administrative tasks similarly
to what HBaseAdmin of HBase provides. We can use kudu executable though,
it's not ideal but it does the job.
To use "kudu tablet change_config add_replica" or "remove_replica",
we need to know the UUID of the tablet server.
2. List of tablets
We can find the list of tablets from `/metrics` endpoint of each tablet
server or from `/dump-entities` endpoint of master. The latter is better
since it also shows which replica is the leader for each tablet, though
I'm not sure if it's supposed to be used by clients other than Cloudera
Manager.
3. List of tablet servers and their UUIDs
"KuduClient.listTabletServers" of Java client returns the list of
servers. But unlike in C++ client, it does not return the UUIDs of the
servers.
We can extract the list of UUIDs by examining
"tablets/replicas/server_uuid" of "/dump-entities". However, we will
miss tablet servers that currently do not serve any replicas.
----
The proposed patch adds "tablet_servers" to "/dump-entities" which
becomes the single point of truth for external tools.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
---
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
2 files changed, 43 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/5977/1
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5977
to look at the new patch set (#2).
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This adds the list of tablet servers to the output of /dump-entities
endpoint of master server to facilitate the development of external
tools and scripts.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
---
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
2 files changed, 58 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/5977/2
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 5: Code-Review+2
Looks good, thanks for the contribution!
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Junegunn Choi has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 3:
Let me add `rpc_addrs` as well.
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has submitted this change and it was merged.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This adds the list of tablet servers to the output of /dump-entities
endpoint of master server to facilitate the development of external
tools and scripts.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Reviewed-on: http://gerrit.cloudera.org:8080/5977
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Kudu Jenkins
---
M src/kudu/integration-tests/delete_table-test.cc
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
3 files changed, 71 insertions(+), 1 deletion(-)
Approvals:
Todd Lipcon: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 1:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/5977/1//COMMIT_MSG
Commit Message:
Line 12: Background:
thanks for the nice background. I think for the final commit message, though, we should remove some of the commentary and just describe what the change does, and perhaps a short explanation of the purpose (so that external tools can easily find a list of which tablet servers to fetch additional metrics from).
PS1, Line 16: tablets across the nodes a la HBase, so we decided to write an external
: tool for that purpopse. In order to do that, we need the followings.
would be great to start a discussion on the dev@ mailing list describing your approach for the balancer tool. This is a commonly requested feature so I imagine we'll want to include one with Kudu as a standard feature in the future, and hearing about your experience would be useful.
PS1, Line 32: since it also shows which replica is the leader for each tablet, though
: I'm not sure if it's supposed to be used by clients other than Cloudera
: Manager.
as a public endpoint, I think we should treat it like a public API that anyone can use, with the normal compatibility guarantees. As an open source project, we shouldn't have private APIs for commercial tools like CM.
PS1, Line 37:
: "KuduClient.listTabletServers" of Java client returns the list of
: servers. But unlike in C++ client, it does not return the UUIDs of the
: servers.
would be nice to have a patch to expand the API as well here. Though I agree that exposing in the JSON endpoint may be handy for people writing scripts.
http://gerrit.cloudera.org:8080/#/c/5977/1/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
PS1, Line 531: if (reg.http_addresses_size() > 0) {
: const HostPortPB& hostPort = reg.http_addresses(0);
: jw.String("addr");
: jw.String(Substitute("$0:$1", hostPort.host(), hostPort.port()));
: }
even though typically we only have one http address per server, I think it would be best to expose the whole array here, and if the client only cares about the first address, put that logic in the client. Otherwise this will be a breaking change later if we want to expose the whole list.
PS1, Line 537: jw.String("live");
: jw.Bool(!desc->PresumedDead());
would it be useful to expose the underlying time since heartbeat too?
I can see the tablet server version being useful as well.
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Junegunn Choi has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 2:
(6 comments)
http://gerrit.cloudera.org:8080/#/c/5977/1//COMMIT_MSG
Commit Message:
Line 12:
> thanks for the nice background. I think for the final commit message, thoug
Agreed, I fixed it. I initially wrote the message with Jira in mind like I used to do with other Apache projects, but later I noticed that Kudu uses Gerrit for patches instead of Jira, and I ended up putting it in the commmit message, but I agree that it's not the right place.
So a related question is: do you create a separate Jira issue beforehand for explaining/discussing the rationale for the change, or just use Gerrit comments?
PS1, Line 16:
:
> would be great to start a discussion on the dev@ mailing list describing yo
Good idea, I'll share my experience and thoughts regarding tooling later.
PS1, Line 32:
:
:
> as a public endpoint, I think we should treat it like a public API that any
I see, note that the endpoint is currently undocumented. I found it in the source code.
PS1, Line 37:
:
:
:
> would be nice to have a patch to expand the API as well here. Though I agre
Yeah, patching the Java client was the first thing I considered, but I wasn't sure whether you would want to add a separate method named `listTabletServerUUIDs` or update the existing `listTabletServers` method to return host/uuid pairs to match the C++ client. The latter will break backward compatibility at the moment but it seems to make more sense in the long run.
http://gerrit.cloudera.org:8080/#/c/5977/1/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
PS1, Line 531:
: jw.String("addrs");
: jw.StartArray();
: for (const HostPortPB& hostPort : reg.http_addresses()) {
:
> even though typically we only have one http address per server, I think it
Done. Changed the label to the plural form `addrs`, let me know if you have other thoughts.
PS1, Line 537: jw.EndArray();
:
> would it be useful to expose the underlying time since heartbeat too?
Good idea. Added `millis_since_heartbeat` (any idea for a better name?). I believe millisecond precision should be sufficient for most use cases. We would still need "live" entry since the external tool may not know the value of `tserver_unresponsive_timeout_ms`.
Also added `version`.
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Junegunn Choi has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5977/4/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
Line 534: for (const HostPortPB& hostPort : reg.rpc_addresses()) {
> sorry, one more style nit I missed: can you name this 'host_port' per our C
Oh, sorry, my oversight.
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5977
to look at the new patch set (#3).
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This adds the list of tablet servers to the output of /dump-entities
endpoint of master server to facilitate the development of external
tools and scripts.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
---
M src/kudu/integration-tests/delete_table-test.cc
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
3 files changed, 63 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/5977/3
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 2:
(5 comments)
Looks like DeleteTableTest has some assertion which needs updating as well
http://gerrit.cloudera.org:8080/#/c/5977/1//COMMIT_MSG
Commit Message:
Line 12:
> Agreed, I fixed it. I initially wrote the message with Jira in mind like I
We don't really have a standard process. Often if there is some pre-discussion to have, we start a mailing list thread on dev@kudu.apache.org. Or, if it's just a quick thought, you might consider joining the kudu slack (https://getkudu-slack.herokuapp.com/) and chatting with devs there. Or, what you did here is fine, too -- can always edit the commit message before submitting.
For true bugs (particularly bugs which we know are present in earlier releases) we do try to file a JIRA first, so that it's easier to refer to when talking with users, etc, but for an improvement like this we aren't strict about 'jira-per-commit' like some other Apache projects.
PS1, Line 32:
:
:
> I see, note that the endpoint is currently undocumented. I found it in the
Yea, that's probably more of an oversight than on purpose :) We do have some docs on /metrics here: https://kudu.apache.org/docs/administration.html#_common_web_interface_pages so it would make sense to document dump-entities in the same place.
PS1, Line 37:
:
:
:
> Yeah, patching the Java client was the first thing I considered, but I wasn
yea, it's a shame that we return List<String> there. Ideally we would return a List<TabletServer> so that it's easy to add information in later versions. I do think since it's a public API we need to avoid breaking that API. Perhaps we can leave 'getTabletServersList()' as is, and @deprecate it, and then add a new 'getTabletServers' which returns a List<TabletServer> with appropriate getters.
http://gerrit.cloudera.org:8080/#/c/5977/2/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
Line 532: jw.String("addrs");
one little nit: can we use 'http_addrs' here so that if we decide to start exposing rpc addrs it's less confusing?
Line 535: jw.String(Substitute("$0:$1", hostPort.host(), hostPort.port()));
one more thought: we now have the ability for tablet servers to speak HTTPS. Should we be using 'http://' here in the string, so that we can later plumb through a flag about whether the TS has SSL enabled and swap in HTTPS in that case? Currently the TSRegistration doesn't hold this info, but should be a good idea to add it (come to think of it we probably need to do this in order for the normal master UI to work properly, too!)
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5977
to look at the new patch set (#5).
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This adds the list of tablet servers to the output of /dump-entities
endpoint of master server to facilitate the development of external
tools and scripts.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
---
M src/kudu/integration-tests/delete_table-test.cc
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
3 files changed, 71 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/5977/5
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Junegunn Choi has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 2:
(3 comments)
I thought it was some kind of a flaky test case, let me look into that.
http://gerrit.cloudera.org:8080/#/c/5977/1//COMMIT_MSG
Commit Message:
PS1, Line 37:
:
:
:
> yea, it's a shame that we return List<String> there. Ideally we would retur
Sounds good to me.
http://gerrit.cloudera.org:8080/#/c/5977/2/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
Line 532: jw.String("addrs");
> one little nit: can we use 'http_addrs' here so that if we decide to start
Oh yeah, that makes more sense. Let me fix it.
Line 535: jw.String(Substitute("$0:$1", hostPort.host(), hostPort.port()));
> one more thought: we now have the ability for tablet servers to speak HTTPS
Agreed.
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5977/4/src/kudu/master/master-path-handlers.cc
File src/kudu/master/master-path-handlers.cc:
Line 534: for (const HostPortPB& hostPort : reg.rpc_addresses()) {
sorry, one more style nit I missed: can you name this 'host_port' per our C++ style guide?
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Add list of tablet servers to /dump-entities
Posted by "Junegunn Choi (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5977
to look at the new patch set (#4).
Change subject: Add list of tablet servers to /dump-entities
......................................................................
Add list of tablet servers to /dump-entities
This adds the list of tablet servers to the output of /dump-entities
endpoint of master server to facilitate the development of external
tools and scripts.
Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
---
M src/kudu/integration-tests/delete_table-test.cc
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/master-test.cc
3 files changed, 71 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/5977/4
--
To view, visit http://gerrit.cloudera.org:8080/5977
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia8ef52a6d2c0c193f5f5db359e144dc1a403a8ab
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Junegunn Choi <ju...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>