Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1834 Don't redact KuduRowResult::ToString() ......................................................................
KUDU-1834 Don't redact KuduRowResult::ToString() A user on the client side calling ToString on the results of a scan will likely expect the data to not be redacted. This patch forces KuduScanPatch::RowPtr::ToString() to not redact. A test case is added to client-test that turns redaction on and compares the output of ToString with a redacted row. If the row is redacted, this case fill fail. Change-Id: If966e9ba76f747de85350ffa2b91fcf05a9ad324 Reviewed-on: http://gerrit.cloudera.org:8080/6222 Reviewed-by: Adar Dembo <a...@cloudera.com> Tested-by: Kudu Jenkins --- M src/kudu/client/client-test.cc M src/kudu/client/scan_batch.cc 2 files changed, 22 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6222 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If966e9ba76f747de85350ffa2b91fcf05a9ad324 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>