[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: consensus: fix some clang-tidy warnings .. consensus: fix some clang-tidy warnings This fixes most of the clang-tidy warnings in the consensus module. I'm

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: consensus: fix some clang-tidy warnings .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/4454 To unsubscribe, visit http://

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: consensus: fix some clang-tidy warnings .. Patch Set 2: the one remaining warning is fixed by the next patch in this series -- To view, visit http://gerrit.cloud

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-23 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: consensus: fix some clang-tidy warnings .. Patch Set 2: still missing some tidy warnings right? -- To view, visit http://gerrit.cloudera.org:8080/4454 To

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-19 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: consensus: fix some clang-tidy warnings .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/4454/1/src/kudu/consensus/log_metrics.h File src/kudu/cons

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-19 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4454 to look at the new patch set (#2). Change subject: consensus: fix some clang-tidy warnings

[kudu-CR] consensus: fix some clang-tidy warnings

2016-09-19 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/4454 to review the following change. Change subject: consensus: fix some clang-tidy warnings .. consensus: fix some