Adar Dembo has posted comments on this change. Change subject: KUDU-1896 (part 2): enable redaction on the web UI ......................................................................
Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6193/1/src/kudu/server/webserver.cc File src/kudu/server/webserver.cc: PS1, Line 307: : If I'm understanding your commit message correctly, as the (complex) redaction patchset evolved, this became no longer true, right? That is, we shouldn't have been concerned about needlessly redacting "useful" metadata like partition keys, table names, column names, etc? -- To view, visit http://gerrit.cloudera.org:8080/6193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4d31f87fd10d177adc2d98dcb049f3bcf6ecdbe2 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
