[kudu-CR] KUDU-2295 fix nullptr dereference in Tablet

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9574 Change subject: KUDU-2295 fix nullptr dereference in Tablet .. KUDU-2295 fix nullptr dereference in Tablet Prior to this patch, in the c

[kudu-CR] KUDU-2322: don't spew logs about behind-log-GC follower

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9573 Change subject: KUDU-2322: don't spew logs about behind-log-GC follower .. KUDU-2322: don't spew logs about behind-log-GC follower This

[kudu-CR] WIP: KUDU-2293 fix cleanup after failed copies

2018-03-09 Thread Andrew Wong (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9452 to look at the new patch set (#3). Change subject: WIP: KUDU-2293 fix cleanup after failed copies .. WIP

[kudu-CR] condition variable: rework timed waiting mechanisms and associated cleanup

2018-03-09 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9572 to review the following change. Change subject: condition_variable: rework timed waiting mechanisms and associated cleanup ...

[kudu-CR] [release notes] rolling upgrade 1.6 -> 1.7 isn't possible

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9571 ) Change subject: [release_notes] rolling upgrade 1.6 -> 1.7 isn't possible .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080

[kudu-CR] [release notes] rolling upgrade 1.6 -> 1.7 isn't possible

2018-03-09 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Mike Percy, Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Grant Henke, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9571 to look at the new patch set (#2). Change subject: [release_notes] rolling upgrade 1.6 -> 1.7 isn't possible ..

[kudu-CR] [release notes] rolling upgrade 1.6 -> 1.7 isn't possible

2018-03-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9571 ) Change subject: [release_notes] rolling upgrade 1.6 -> 1.7 isn't possible .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. [itests] update on raft_consensus_stress-itest Updates on raf

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has removed Kudu Jenkins from this change. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. Removed reviewer Kudu Jenkins with the following votes: * Ver

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. Patch Set 2: Verified+1 Unrelated flakes in RemoteKsckTest.TestChec

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org

[kudu-CR] [release notes] rolling upgrade 1.6 -> 1.7 isn't possible

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9571 Change subject: [release_notes] rolling upgrade 1.6 -> 1.7 isn't possible .. [release_notes] rolling upgrade 1.6 -> 1.7 isn't possible C

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/9567/1

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9567 to look at the new patch set (#2). Change subject: [itests] update on raft_consensus_stress-itest ...

[kudu-CR] WIP: KUDU-2293 fix cleanup after failed copies

2018-03-09 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9452 ) Change subject: WIP: KUDU-2293 fix cleanup after failed copies .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/9452/2//C

[kudu-CR] WIP: KUDU-2293 fix cleanup after failed copies

2018-03-09 Thread Andrew Wong (Code Review)
Hello Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9452 to look at the new patch set (#2). Change subject: WIP: KUDU-2293 fix cleanup after failed copies .. WIP

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9567 ) Change subject: [itests] update on raft_consensus_stress-itest .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/9567/1/sr

[kudu-CR] [itests] update on raft consensus stress-itest

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9567 Change subject: [itests] update on raft_consensus_stress-itest .. [itests] update on raft_consensus_stress-itest Updates on raft_consens

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9565 ) Change subject: [master] add feature flag for 3-4-3 scheme .. [master] add feature flag for 3-4-3 scheme This changelist addres

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9565 ) Change subject: [master] add feature flag for 3-4-3 scheme .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:80

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Alexey Serbin (Code Review)
Hello Mike Percy, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9565 to look at the new patch set (#3). Change subject: [master] add feature flag for 3-4-3 scheme ..

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9565 ) Change subject: [master] add feature flag for 3-4-3 scheme .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/9565/2/src

[kudu-CR](branch-1.6.x) [tablet] fix nullptr dereference while capturing iterators

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9549 ) Change subject: [tablet] fix nullptr dereference while capturing iterators .. [tablet] fix nullptr dereference while capturing i

[kudu-CR](branch-1.6.x) KUDU-2274. RaftConsensus should not access cmeta when shutdown

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9548 ) Change subject: KUDU-2274. RaftConsensus should not access cmeta when shutdown .. KUDU-2274. RaftConsensus should not access cme

[kudu-CR](branch-1.6.x) [tablet service] remove useless call

2018-03-09 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9550 ) Change subject: [tablet_service] remove useless call .. [tablet_service] remove useless call Removed useless call of TabletRepl

[kudu-CR](branch-1.6.x) [tablet service] remove useless call

2018-03-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9550 ) Change subject: [tablet_service] remove useless call .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/955

[kudu-CR](branch-1.6.x) [tablet] fix nullptr dereference while capturing iterators

2018-03-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9549 ) Change subject: [tablet] fix nullptr dereference while capturing iterators .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit

[kudu-CR](branch-1.6.x) KUDU-2274. RaftConsensus should not access cmeta when shutdown

2018-03-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9548 ) Change subject: KUDU-2274. RaftConsensus should not access cmeta when shutdown .. Patch Set 2: Code-Review+2 -- To view, visit http://ge

[kudu-CR] KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect

2018-03-09 Thread Grant Henke (Code Review)
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9559 ) Change subject: KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect .. KUDU-2093: Document AbstractKuduScannerBuil

[kudu-CR] KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect

2018-03-09 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/9559 ) Change subject: KUDU-2093: Document AbstractKuduScannerBuilder limit has no effect .. Patch Set 1: Code-Review+2 -- To view, vi

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9565 ) Change subject: [master] add feature flag for 3-4-3 scheme .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/9565/2/src/ku

[kudu-CR] [master] add feature flag for 3-4-3 scheme

2018-03-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9565 ) Change subject: [master] add feature flag for 3-4-3 scheme .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/9565/2/src/k