Hello Tidy Bot, Mike Percy, Alexey Serbin, Attila Bukor, Andrew Wong, Adar
Dembo, Grant Henke,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11665
to look at the new patch set (#8).
Change subject: [tools] Add a describe table tool
......................................................................
[tools] Add a describe table tool
This adds a very basic tool to describe a table, printing out its
schema, its partition schema, the current range partitions, and the
replication factor. It eschews completeness for brevity, leaving out,
for example, column storage attributes, since they clutter the output
and make it hard to read. My use case for this tool is as an easy way a
user can gather the fundamental info about a table and show it to
someone else. It should be easy to copy and paste, usually fit on one
line in a terminal window, and not depend on fancy formatting or a
fixed-width font to display correctly.
Sample output:
$ kudu table describe localhost:7053
default.loadgen_auto_f6d439ab21e4408486786289297f7db0
TABLE default.loadgen_auto_f6d439ab21e4408486786289297f7db0 (
key INT64 NOT NULL,
int_val INT32 NULLABLE,
string_val STRING NULLABLE,
PRIMARY KEY (key)
)
HASH (key) PARTITIONS 2,
RANGE (key) (
PARTITION UNBOUNDED
)
REPLICAS 1
Getting this "SQL-ish" look required changing how schemas and column
schemas are stringified, so there are some knock-on changes in various
tests as well.
I also reorganized the order of `kudu table` commands so they are in
alphabetical order.
Change-Id: Ib02f2a94cf11d0e60b32cd85be920ca9b99dd977
---
M src/kudu/client/client-test.cc
M src/kudu/client/client-unittest.cc
M src/kudu/client/client.h
M src/kudu/client/schema.cc
M src/kudu/common/partial_row-test.cc
M src/kudu/common/row_changelist-test.cc
M src/kudu/common/row_operations-test.cc
M src/kudu/common/schema-test.cc
M src/kudu/common/schema.cc
M src/kudu/tablet/tablet_bootstrap-test.cc
M src/kudu/tools/CMakeLists.txt
M src/kudu/tools/kudu-admin-test.cc
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_common.cc
M src/kudu/tools/tool_action_common.h
M src/kudu/tools/tool_action_table.cc
M src/kudu/tserver/tablet_server-test.cc
17 files changed, 375 insertions(+), 130 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/65/11665/8
--
To view, visit http://gerrit.cloudera.org:8080/11665
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib02f2a94cf11d0e60b32cd85be920ca9b99dd977
Gerrit-Change-Number: 11665
Gerrit-PatchSet: 8
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
