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 <[email protected]> Tested-by: Kudu Jenkins (cherry picked from commit f1da1eb450cb77878d44d1115d64c05f652378f1) Reviewed-on: http://gerrit.cloudera.org:8080/6244 Reviewed-by: Todd Lipcon <[email protected]> --- M src/kudu/client/client-test.cc M src/kudu/client/scan_batch.cc 2 files changed, 22 insertions(+), 0 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6244 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: If966e9ba76f747de85350ffa2b91fcf05a9ad324 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-1.3.x Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
