[kudu-CR] WIP: KUDU-1812. redaction of PB fields

2016-12-22 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: WIP: KUDU-1812. redaction of PB fields .. Patch Set 2: (8 comments) http://gerrit.cloudera.org:8080/#/c/5553/2/src/kudu/util/pb_util.cc File src/kudu/util/pb_util

[kudu-CR] WIP: KUDU-1812. redaction of PB fields

2016-12-21 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: WIP: KUDU-1812. redaction of PB fields .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5553/2/src/kudu/util/pb_util.proto File src/kudu/util/pb_ut

[kudu-CR] WIP: KUDU-1812. redaction of PB fields

2016-12-21 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: WIP: KUDU-1812. redaction of PB fields .. Patch Set 2: (6 comments) I did a pass over the .proto files and I think you've nailed everything. The most important is

[kudu-CR] WIP: KUDU-1812. redaction of PB fields

2016-12-21 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5553 to look at the new patch set (#2). Change subject: WIP: KUDU-1812. redaction of PB fields .. WIP: KUDU-1812. redact

[kudu-CR] WIP: KUDU-1812. redaction of PB fields

2016-12-20 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5553 to review the following change. Change subject: WIP: KUDU-1812. redaction of PB fields .. WIP: KUD