Zoltan Chovan has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21496 )

Change subject: Include table UUID when using --show_table_info
......................................................................


Patch Set 5:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/21496/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/21496/3//COMMIT_MSG@7
PS3, Line 7: table UUID when using --show_table_info
> Maybe, it's time to make `kudu table list` start accepting `--columns` opti
I made the recommended changes. Should I already mark the '--show_table_info' 
as deprecated to is that something that needs to be done at the next release?


http://gerrit.cloudera.org:8080/#/c/21496/4/src/kudu/tools/tool_action_table.cc
File src/kudu/tools/tool_action_table.cc:

http://gerrit.cloudera.org:8080/#/c/21496/4/src/kudu/tools/tool_action_table.cc@340
PS4, Line 340:         }
> what happens if I set --columns=id,nameeeee ?
Done



--
To view, visit http://gerrit.cloudera.org:8080/21496
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I324b920e6feb6139e7d884e3cf08069b0cb922a4
Gerrit-Change-Number: 21496
Gerrit-PatchSet: 5
Gerrit-Owner: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
Gerrit-Comment-Date: Thu, 13 Jun 2024 12:27:34 +0000
Gerrit-HasComments: Yes

Reply via email to