Hello Kudu Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/19767

to look at the new patch set (#2).

Change subject: [tool] add details show for not fully quiesced tserver
......................................................................

[tool] add details show for not fully quiesced tserver

We can run `kudu tserver quiesce start` with the
'--error_if_not_fully_quiesced' option to indicating
that all leaders have been moved away from the tablet
server and all on-going scans have completed for now.

However, in some cases, such as the presence of a
single replica table, it may result in quiesce operations
not being completed. If the total number of tables is
very large, we will not be able to easily locate the root
cause of quiesce jamming.

To solve this issue, I have added a parameter
'--show_details_if_not_fully_quiesced' to the quiesce
progress query, which in conjunction with parameter
'--error_if_not_fully_quiesced', can easily determine which
tablets or scanners are present, blocking the completion of
the query operation.

At the same time, I added test to ensure that the new
parameter will take effect.

Change-Id: I1dde33f1c36831b62c321eeea2040bc4a5ac21d0
---
M src/kudu/consensus/raft_consensus.cc
M src/kudu/consensus/raft_consensus.h
M src/kudu/integration-tests/tablet_server_quiescing-itest.cc
M src/kudu/integration-tests/ts_tablet_manager-itest.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_tserver.cc
M src/kudu/tserver/scanners-test.cc
M src/kudu/tserver/scanners.cc
M src/kudu/tserver/scanners.h
M src/kudu/tserver/tablet_service.cc
M src/kudu/tserver/ts_tablet_manager.cc
M src/kudu/tserver/ts_tablet_manager.h
M src/kudu/tserver/tserver_admin.proto
13 files changed, 218 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/67/19767/2
--
To view, visit http://gerrit.cloudera.org:8080/19767
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1dde33f1c36831b62c321eeea2040bc4a5ac21d0
Gerrit-Change-Number: 19767
Gerrit-PatchSet: 2
Gerrit-Owner: KeDeng <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to