Will Berkeley has posted comments on this change. (
http://gerrit.cloudera.org:8080/11665 )
Change subject: [tools] Add a describe table tool
......................................................................
Patch Set 5:
Yeah, we can do that. Though, re: the CREATE TABLE on /table, we went a
different direction so that's only what you need to establish an external table
over the existing table:
CREATE EXTERNAL TABLE `<placeholder>` STORED AS KUDU
TBLPROPERTIES(
'kudu.table_name' =
'default.loadgen_auto_f6d439ab21e4408486786289297f7db0',
'kudu.master_addresses' =
'127.0.0.1:7053,127.0.0.1:7052,127.0.0.1:7051')
Do you prefer the table or SQL syntax? I think I prefer the latter because it
is more compact. It is more likely to fit when printed in a terminal window and
easier to follow when it wraps.
I don't want to add anything that references or evokes impala specifically,
though, like the TBLPROPERTIES or STORED AS KUDU parts. This is meant to be a
useful little tool for working directly with Kudu, esp if there's no impala.
--
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: comment
Gerrit-Change-Id: Ib02f2a94cf11d0e60b32cd85be920ca9b99dd977
Gerrit-Change-Number: 11665
Gerrit-PatchSet: 5
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]>
Gerrit-Comment-Date: Fri, 12 Oct 2018 22:13:52 +0000
Gerrit-HasComments: No
