You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Wang Xixu (Code Review)" <ge...@cloudera.org> on 2022/12/06 03:27:29 UTC

[kudu-CR] [KUDU-3426] Ignore showing parition information

Hello Kudu Jenkins, 

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

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

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

Change subject: [KUDU-3426] Ignore showing parition information
......................................................................

[KUDU-3426] Ignore showing parition information

The field: partition_key_start/partition_key_end in PartitionPB
is bytes. When decoding the result of ksck in json format may fail.
In a machine, it needs to use latin1 to decode the result, some
needs to use cp1252, the other use utf8. Most time, it is no
need to fetch partition information. Therefore using this flag
to skip showing partition information.

Change-Id: Ieb48a9a5e41614e617d3fdbe73da10c8b92a6bc1
---
M src/kudu/tools/ksck-test.cc
M src/kudu/tools/ksck.cc
M src/kudu/tools/ksck_results.cc
M src/kudu/tools/tool_action_cluster.cc
4 files changed, 29 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/20/19320/3
-- 
To view, visit http://gerrit.cloudera.org:8080/19320
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ieb48a9a5e41614e617d3fdbe73da10c8b92a6bc1
Gerrit-Change-Number: 19320
Gerrit-PatchSet: 3
Gerrit-Owner: Wang Xixu <14...@qq.com>
Gerrit-Reviewer: Kudu Jenkins (120)