[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 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/3648 to look at the new patch set (#9). Change subject: KUDU-1311 [master] support adding and dropping range partitions

[kudu-CR] Remove ASF incubation callouts

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Remove ASF incubation callouts .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2573/ -- To view, visit http://gerrit.cloudera.org:8080/3699

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-07-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: client/sample.cc: fixed a couple of crashes .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/3685/1//COMMIT_MSG Commit Message: Line 11: while

[kudu-CR] Remove ASF incubation callouts

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3699 to review the following change. Change subject: Remove ASF incubation callouts .. Remove ASF incubation callouts Now

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 8: (1 comment)

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2574/ -- To

[kudu-CR] Add Apache Kudu logo to the source tree

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add Apache Kudu logo to the source tree .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2575/ -- To view, visit

[kudu-CR] Add Apache Kudu logo to the source tree

2016-07-20 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3701 to review the following change. Change subject: Add Apache Kudu logo to the source tree .. Add Apache Kudu logo to

[kudu-CR] [java client] Integrate with the replay cache

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 5: (1 comment)

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/2570/ -- To view, visit

[kudu-CR] C++ client: fix on KuduSession::GetPendingErrors()

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: C++ client: fix on KuduSession::GetPendingErrors() .. C++ client: fix on KuduSession::GetPendingErrors() If re-using the same std::vector instance as a

[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: delete_table-test: bump timeout waiting for tablet to start .. Patch Set 1: Code-Review+2 self-+2ing trivial patch -- To view, visit

[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: delete_table-test: bump timeout waiting for tablet to start .. delete_table-test: bump timeout waiting for tablet to start It seems like

[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: delete_table-test: bump timeout waiting for tablet to start .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2569/ -- To view, visit

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3632 to look at the new patch set (#3). Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial)

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2571/ -- To

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 9: (1 comment)

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2572/ -- To view, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 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/3648 to look at the new patch set (#10). Change subject: KUDU-1311 [master] support adding and dropping range partitions

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 9: (3 comments)

[kudu-CR] [java client] Integrate with the replay cache

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Integrate with the replay cache .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3631 To unsubscribe, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 9: (2 comments)

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/3667/2//COMMIT_MSG

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 10: Build Started http://104.196.14.100/job/kudu-gerrit/2576/ -- To view, visit

[kudu-CR] Doxygen for C++ client API

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Doxygen for C++ client API .. Patch Set 12: OK that's all my feedback. Thanks again for taking this on! -- To view, visit http://gerrit.cloudera.org:8080/3619

[kudu-CR] KUDU-1358 (part 2): heartbeat to every master

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1358 (part 2): heartbeat to every master .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3610 To unsubscribe, visit

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-1358 (part 1): master should accept heartbeat even if follower

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower .. Patch Set 10: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3609 To

[kudu-CR] docs: Remove reference to old kudu-user mailing list

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: docs: Remove reference to old kudu-user mailing list .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3711 To unsubscribe, visit

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 10: (3 comments)

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3055 to look at the new patch set (#10). Change subject: [java client] Redo how we manage exceptions ..

[kudu-CR] ksck: multi-thread the fetching of replica info from tablet servers

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3705 to review the following change. Change subject: ksck: multi-thread the fetching of replica info from tablet servers

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 11: (1 comment)

[kudu-CR] Doxygen for C++ client API

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Doxygen for C++ client API .. Patch Set 12: (19 comments) http://gerrit.cloudera.org:8080/#/c/3619/12/src/kudu/client/client.h File src/kudu/client/client.h:

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. Patch Set 6: (1 comment)

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 12: Build Started http://104.196.14.100/job/kudu-gerrit/2583/ -- To view, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3055 to look at the new patch set (#12). Change subject: [java client] Redo how we manage exceptions ..

[kudu-CR](gh-pages) www: Remove "incubating" from web site

2016-07-20 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3709 to review the following change. Change subject: www: Remove "incubating" from web site .. www: Remove "incubating"

[kudu-CR](gh-pages) www: Replace old Kudu logo with new responsive Apache Kudu logo

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Replace old Kudu logo with new responsive Apache Kudu logo .. Patch Set 1: Rendered at http://mpercy.github.io/kudu/ -- To view, visit

[kudu-CR] ksck: fix a crash in checksum mode on tables with many tablets

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: fix a crash in checksum mode on tables with many tablets .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2588/ -- To view, visit

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3715 to review the following change. Change subject: ksck: improve output for long-running ksck checksums

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: improve output for long-running ksck checksums .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2589/ -- To view, visit

[kudu-CR] ksck: fix a crash in checksum mode on tables with many tablets

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3714 to review the following change. Change subject: ksck: fix a crash in checksum mode on tables with many tablets

[kudu-CR] master: do not delete unknown tablets

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 6: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3645 To unsubscribe, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2582/ -- To view, visit

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 11: (1 comment)

[kudu-CR](gh-pages) www: Replace old Kudu logo with new responsive Apache Kudu logo

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: www: Replace old Kudu logo with new responsive Apache Kudu logo .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3708 To

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1311 [master] support adding and dropping range partitions .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2586/ -- To view, visit

[kudu-CR] KUDU-1311 [master] support adding and dropping range partitions

2016-07-20 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/3648 to look at the new patch set (#11). Change subject: KUDU-1311 [master] support adding and dropping range partitions

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2587/ -- To view,

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Dinesh Bhat (Code Review)
Hello Mike Percy, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3667 to look at the new patch set (#4). Change subject: KUDU-1492: Show column encodings/compression on table page in master

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3667 To

[kudu-CR] KUDU-1492: Show column encodings/compression on table page in master

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: KUDU-1492: Show column encodings/compression on table page in master .. KUDU-1492: Show column encodings/compression on table page in master This

[kudu-CR] ksck: report hostnames instead of IP addresses for tablet servers

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3704 to review the following change. Change subject: ksck: report hostnames instead of IP addresses for tablet servers

[kudu-CR] ksck: multi-thread the fetching of replica info from tablet servers

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: multi-thread the fetching of replica info from tablet servers .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2578/ -- To view,

[kudu-CR] ksck: report hostnames instead of IP addresses for tablet servers

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: report hostnames instead of IP addresses for tablet servers .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2577/ -- To view, visit

[kudu-CR] ksck: report hostnames instead of IP addresses for tablet servers

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3704 to look at the new patch set (#2). Change subject: ksck: report hostnames instead of IP addresses for tablet servers

[kudu-CR] ksck: also print info about the good replicas for a bad tablet

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: also print info about the good replicas for a bad tablet .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2581/ -- To view, visit

[kudu-CR] ksck: also print info about the good replicas for a bad tablet

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3706 to review the following change. Change subject: ksck: also print info about the good replicas for a bad tablet

[kudu-CR] ksck: multi-thread the fetching of replica info from tablet servers

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: ksck: multi-thread the fetching of replica info from tablet servers .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2579/ -- To view,

[kudu-CR] docs: Remove reference to old kudu-user mailing list

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: docs: Remove reference to old kudu-user mailing list .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2584/ -- To view, visit

[kudu-CR](gh-pages) www: Remove "incubating" from web site

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Remove "incubating" from web site .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3709 To unsubscribe, visit

[kudu-CR](gh-pages) www: Replace old Kudu logo with new responsive Apache Kudu logo

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: www: Replace old Kudu logo with new responsive Apache Kudu logo .. www: Replace old Kudu logo with new responsive Apache Kudu logo Change-Id:

[kudu-CR](gh-pages) www: Remove references to old Google Groups mailing list

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: www: Remove references to old Google Groups mailing list .. www: Remove references to old Google Groups mailing list We missed some references when first

[kudu-CR] docs: Remove reference to old kudu-user mailing list

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: docs: Remove reference to old kudu-user mailing list .. docs: Remove reference to old kudu-user mailing list Change-Id:

[kudu-CR](gh-pages) www: Remove references to old Google Groups mailing list

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Remove references to old Google Groups mailing list .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/3710 To unsubscribe, visit

[kudu-CR](gh-pages) www: Remove "incubating" from web site

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. Change subject: www: Remove "incubating" from web site .. www: Remove "incubating" from web site Also refer to the mailing lists as being at @kudu.apache.org TODO later

[kudu-CR](gh-pages) www: Replace old Kudu logo with new responsive Apache Kudu logo

2016-07-20 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3708 to review the following change. Change subject: www: Replace old Kudu logo with new responsive Apache Kudu logo ..

[kudu-CR](gh-pages) www: Remove "incubating" from web site

2016-07-20 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: www: Remove "incubating" from web site .. Patch Set 1: Rendered at http://mpercy.github.io/kudu/ -- To view, visit http://gerrit.cloudera.org:8080/3709 To

[kudu-CR] [java client] Redo how we manage exceptions

2016-07-20 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [java client] Redo how we manage exceptions .. Patch Set 12: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3055 To unsubscribe, visit

[kudu-CR] ksck: multi-thread the fetching of replica info from tablet servers

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: multi-thread the fetching of replica info from tablet servers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3705

[kudu-CR] ksck: fix a crash in checksum mode on tables with many tablets

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: fix a crash in checksum mode on tables with many tablets .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3714 To

[kudu-CR] ksck: also print info about the good replicas for a bad tablet

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: also print info about the good replicas for a bad tablet .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3706 To

[kudu-CR] master: do not delete unknown tablets

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/2599/ -- To view, visit

[kudu-CR] ksck: improve output for long-running ksck checksums

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: improve output for long-running ksck checksums .. Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3715/1//COMMIT_MSG

[kudu-CR] ksck: improve filtering capability

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Jean-Daniel Cryans, Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3716 to review the following change. Change subject: ksck: improve filtering capability .. ksck:

[kudu-CR] WIP: KUDU-1536. ksck checksum scan crashes if a replica is on a server not known to master

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: WIP: KUDU-1536. ksck checksum scan crashes if a replica is on a server not known to master .. Patch Set 1: Build Started

[kudu-CR] ksck: improve filtering capability

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: improve filtering capability .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3716 To unsubscribe, visit

[kudu-CR] KUDU-1358 (part 1): master should accept heartbeat even if follower

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2594/ -- To view,

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 3): new multi-master stress test .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2595/ -- To view, visit

[kudu-CR] KUDU-1358 (part 1): master should accept heartbeat even if follower

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3609 to look at the new patch set (#11). Change subject: KUDU-1358 (part 1): master should accept heartbeat even if follower

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3643 to look at the new patch set (#7). Change subject: KUDU-1374: send full tablet report when new leader master is detected

[kudu-CR] KUDU-1358 (part 2): heartbeat to every master

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3610 to look at the new patch set (#11). Change subject: KUDU-1358 (part 2): heartbeat to every master

[kudu-CR] KUDU-1516 ksck should check for more raft-related status issues (partial)

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1516 ksck should check for more raft-related status issues (partial) .. Patch Set 4: (3 comments)

[kudu-CR] master: do not delete unknown tablets

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: do not delete unknown tablets .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2592/ -- To view, visit

[kudu-CR] ksck: report hostnames instead of IP addresses for tablet servers

2016-07-20 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: ksck: report hostnames instead of IP addresses for tablet servers .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3704 To

[kudu-CR] KUDU-1538 (WIP): prevent LBM block ID reuse to avoid potential data loss

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1538 (WIP): prevent LBM block ID reuse to avoid potential data loss .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2600/ -- To view,

[kudu-CR] KUDU-1538: prevent block ID reuse to avoid potential data loss

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1538: prevent block ID reuse to avoid potential data loss .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2601/ -- To view, visit

[kudu-CR] KUDU-1538: prevent block ID reuse to avoid potential data loss

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3719 to look at the new patch set (#2). Change subject: KUDU-1538: prevent block ID reuse to avoid potential data loss

[kudu-CR] master: do not delete unknown tablets

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3645 to look at the new patch set (#8). Change subject: master: do not delete unknown tablets .. master:

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: client/sample.cc: fixed a couple of crashes .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3685/1//COMMIT_MSG Commit Message: Line 11: while

[kudu-CR] client/sample.cc: fixed a couple of crashes

2016-07-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: client/sample.cc: fixed a couple of crashes .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/3685/1//COMMIT_MSG Commit Message: Line 11: while

[kudu-CR] tablet peer: update status message on failure to start

2016-07-20 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: tablet_peer: update status message on failure to start .. tablet_peer: update status message on failure to start If the tablet peer fails to start up, we

[kudu-CR] delete table-test: bump timeout waiting for tablet to start

2016-07-20 Thread Todd Lipcon (Code Review)
Hello Mike Percy, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3689 to review the following change. Change subject: delete_table-test: bump timeout waiting for tablet to start ..

[kudu-CR] KUDU-1374: send full tablet report when new leader master is detected

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1374: send full tablet report when new leader master is detected .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/2593/ -- To view,

[kudu-CR] master: do not delete unknown tablets

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3645 to look at the new patch set (#7). Change subject: master: do not delete unknown tablets

[kudu-CR] KUDU-1358 (part 2): heartbeat to every master

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: KUDU-1358 (part 2): heartbeat to every master .. Patch Set 11: Build Started http://104.196.14.100/job/kudu-gerrit/2596/ -- To view, visit

[kudu-CR] c++ client: use operation timeout as deadline for finding new leader master

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3718 to review the following change. Change subject: c++ client: use operation timeout as deadline for finding new leader master

[kudu-CR] master: additional leader lock assertions in catalog manager

2016-07-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: master: additional leader lock assertions in catalog manager .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2597/ -- To view, visit

[kudu-CR] KUDU-1358 (part 3): new multi-master stress test

2016-07-20 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3611 to look at the new patch set (#11). Change subject: KUDU-1358 (part 3): new multi-master stress test

  1   2   >