[kudu-CR] [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

2018-01-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9126 ) Change subject: [tests] de-flaking DontEvictIfRemainingConfigIsUnstable .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#

[kudu-CR] [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

2018-01-25 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/9126 to look at the new patch set (#2). Change subject: [tests] de-flaking DontEvictIfRemainingConfigIsUnstable .

[kudu-CR] Simplify OpId/Timestamp assignment and make it atomic

2018-01-25 Thread David Ribeiro Alves (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7221 to look at the new patch set (#10). Change subject: Simplify OpId/Timestamp assignment and make it atomic

[kudu-CR] KUDU-1927: no half-baked responses on ConnectToMaster

2018-01-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9052 ) Change subject: KUDU-1927: no half-baked responses on ConnectToMaster .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.clou

[kudu-CR] [tests] de-flaking DontEvictIfRemainingConfigIsUnstable

2018-01-25 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/9126 ) Change subject: [tests] de-flaking DontEvictIfRemainingConfigIsUnstable .. Patch Set 1: (2 comments) looks good http://gerrit.cloudera.o

[kudu-CR] back out cblock nomenclature

2018-01-25 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9144 ) Change subject: back out cblock nomenclature .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9144 To uns

[kudu-CR] back out cblock nomenclature

2018-01-25 Thread Dan Burkert (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9144 to review the following change. Change subject: back out cblock nomenclature .. back out cblock nomenclature Change-I

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9137 ) Change subject: internal_mini_cluster: support Cluster/LogVerifier .. Patch Set 5: Just pushed some minor formatting changes and updated

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Mike Percy, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9137 to look at the new patch set (#6). Change subject: internal_mini_cluster: support Cluster/LogVerifier .

[kudu-CR] KUDU-1927: no half-baked responses on ConnectToMaster

2018-01-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9052 ) Change subject: KUDU-1927: no half-baked responses on ConnectToMaster .. Patch Set 8: (3 comments) http://gerrit.cloudera.org:8080/#/c

[kudu-CR] KUDU-1927: no half-baked responses on ConnectToMaster

2018-01-25 Thread Alexey Serbin (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9052 to look at the new patch set (#9). Change subject: KUDU-1927: no half-baked responses on ConnectToMaster

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9137 to look at the new patch set (#5). Change subject: internal_mini_cluster: support Cluster/LogVerifier .. i

[kudu-CR] KUDU-1927: no half-baked responses on ConnectToMaster

2018-01-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/9052 ) Change subject: KUDU-1927: no half-baked responses on ConnectToMaster .. Patch Set 8: (3 comments) http://gerrit.cloudera.org:8080/#/c/9

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9136 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/9136/3

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9136 to look at the new patch set (#4). Change subject: KUDU-1613: evict replicas with wrong server UUIDs ...

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9136 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/9136

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9137 to look at the new patch set (#4). Change subject: internal_mini_cluster: support Cluster/LogVerifier .. i

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9136 to look at the new patch set (#3). Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. KU

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9137 ) Change subject: internal_mini_cluster: support Cluster/LogVerifier .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/9137

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9137 to look at the new patch set (#3). Change subject: internal_mini_cluster: support Cluster/LogVerifier .. i

[kudu-CR] docs: fix number of number masters

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9091 ) Change subject: docs: fix number of number masters .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/9091/3/docs/installat

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9137 to look at the new patch set (#2). Change subject: internal_mini_cluster: support Cluster/LogVerifier .. i

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9136 to look at the new patch set (#2). Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. KU

[kudu-CR] KUDU-1613: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/9136 ) Change subject: KUDU-1613: evict replicas with wrong server UUIDs .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/9136/

[kudu-CR] internal mini cluster: support Cluster/LogVerifier

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9137 Change subject: internal_mini_cluster: support Cluster/LogVerifier .. internal_mini_cluster: support Cluster/LogVerifier This patch encaps

[kudu-CR] KUDU-1620: evict replicas with wrong server UUIDs

2018-01-25 Thread Andrew Wong (Code Review)
Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9136 Change subject: KUDU-1620: evict replicas with wrong server UUIDs .. KUDU-1620: evict replicas with wrong server UUIDs Currently, if a tab

[kudu-CR] design-docs: improve cfile.md

2018-01-25 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/8860 ) Change subject: design-docs: improve cfile.md .. Patch Set 3: I'm going to land this for now, with the intention to update it soon if I c

[kudu-CR] design-docs: improve cfile.md

2018-01-25 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8860 ) Change subject: design-docs: improve cfile.md .. design-docs: improve cfile.md I've continually found the CFile abstraction diffi

[kudu-CR] client: fix UBSAN error in failover calculation

2018-01-25 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9132 ) Change subject: client: fix UBSAN error in failover calculation .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera