[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Add --redact=none flag option for specifying no redaction .. Add --redact=none flag option for specifying no redaction It's still possible to specify no r

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6590 To unsubscribe, visit h

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 4: The differences is that the flags are now getting reset back to the default value every

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 4: Code-Review+2 I'm curious as to how the diff between PS4 and PS3 fixed the test failures.

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6590 to look at the new patch set (#4). Change subject: Add --redact=none flag option for specifying no redaction ...

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 3: The change looks good to me, but can you check why the tests are failing? It looks like rela

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6590 To unsubscribe, visi

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6590 to look at the new patch set (#3). Change subject: Add --redact=none flag option for specifying no redaction .. Add

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6590/1/src/kudu/util/flags.cc File src/

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: Add --redact=none flag option for specifying no redaction .. Add --redact=none flag option for specifying no redaction It's still possible to specify no redaction vi

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Dan Burkert (Code Review)
Hello Hao Hao, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/6590 to review the following change. Change subject: Add --redact=none flag option for specifying no redaction ..