[kudu-CR] [tools] Fix doc'd endianness for kudu test mini cluster

2017-11-09 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/8504 ) Change subject: [tools] Fix doc'd endianness for kudu test mini_cluster .. Patch Set 1: (1 comment)

[kudu-CR] [tools] Fix doc'd endianness for kudu test mini cluster

2017-11-09 Thread Will Berkeley (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8504 to look at the new patch set (#2). Change subject: [tools] Fix doc'd endianness for kudu test mini_cluster

[kudu-CR] [blocking ops] remove extra Socket::IsNonBlocking call

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8454 ) Change subject: [blocking_ops] remove extra Socket::IsNonBlocking call .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] [blocking ops] remove extra Socket::IsNonBlocking call

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8454 ) Change subject: [blocking_ops] remove extra Socket::IsNonBlocking call .. [blocking_ops] remove extra Socket::IsNonBlocking call

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. stopwatch: avoid clang use-after-move

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has removed a vote on this change. Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Patch Set 3: Code-Review+2 -- To view, visit

[kudu-CR] stopwatch: avoid clang use-after-move warnings in LOG TIMING macros

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8502 ) Change subject: stopwatch: avoid clang use-after-move warnings in LOG_TIMING macros .. Patch Set 3: Verified+1 unrelated flake -- To

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 3: (7 comments)

[kudu-CR] clang tidy gerrit: don't require 'requests' module in local mode

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8501 ) Change subject: clang_tidy_gerrit: don't require 'requests' module in local mode .. clang_tidy_gerrit: don't require 'requests'

[kudu-CR] clang tidy gerrit: don't require 'requests' module in local mode

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8501 ) Change subject: clang_tidy_gerrit: don't require 'requests' module in local mode .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [tools] Fix doc'd endianness for kudu test mini cluster

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8504 ) Change subject: [tools] Fix doc'd endianness for kudu test mini_cluster .. [tools] Fix doc'd endianness for kudu test

[kudu-CR] [tools] Fix doc'd endianness for kudu test mini cluster

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8504 ) Change subject: [tools] Fix doc'd endianness for kudu test mini_cluster .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8477 to look at the new patch set (#4). Change subject: [tests] fix flake in RaftConsensusNonVoterITest

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 3: (7 comments)

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Removed reviewer Kudu Jenkins with the following votes: *

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 4: Verified+1 unrelated flake (TSAN warnings in

[kudu-CR] tool: new action for adding to the set of data directories

2017-11-09 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded a new patch set (#9) to the change originally created by Adar Dembo. ( http://gerrit.cloudera.org:8080/8352 ) Change subject: tool: new action for adding to the set of data directories .. tool: new

[kudu-CR] tool: new action for adding to the set of data directories

2017-11-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/8352 ) Change subject: tool: new action for adding to the set of data directories .. Patch Set 9: (8 comments) Did a light refactor in

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. [tests] fix flake in RaftConsensusNonVoterITest Fixed

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 4: Code-Review+2 (1 comment)

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8477 to look at the new patch set (#5). Change subject: [tests] fix flake in RaftConsensusNonVoterITest ..

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 5: (1 comment)

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8477 to look at the new patch set (#6). Change subject: [tests] fix flake in RaftConsensusNonVoterITest

[kudu-CR] [tests] fix flake in RaftConsensusNonVoterITest

2017-11-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/8477 ) Change subject: [tests] fix flake in RaftConsensusNonVoterITest .. Patch Set 6: Code-Review+2 -- To view, visit