Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12976
to look at the new patch set (#2).
Change subject: tool: fixes for kudu local_replica dump rowset
......................................................................
tool: fixes for kudu local_replica dump rowset
This patch makes several adjustments to 'kudu local_replica dump rowset':
- The existing 'metadata_only' and 'nrows' controls were being ignored.
- The existing 'rowset_index' control wasn't working properly.
- I changed the "what to dump" contols to 'dump_all_columns' and
'dump_metadata'. When 'dump_all_columns' is false, the row keys are dumped
in a format that's comparable and ASCII-compatible (currently hex).
This functionality helped me dump a tablet's keys (grouped by rowset), which
I then used for a series of MergeIterator experiments.
Change-Id: Ib50ab4e7b2aa0fec60ce0718d16823945a05cb7f
---
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_local_replica.cc
2 files changed, 161 insertions(+), 57 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/12976/2
--
To view, visit http://gerrit.cloudera.org:8080/12976
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib50ab4e7b2aa0fec60ce0718d16823945a05cb7f
Gerrit-Change-Number: 12976
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Will Berkeley <[email protected]>
