[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-12 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 10: Filed a cpplint issue @ https://github.com/google/styleguide/issues/176 -- To view, vis

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Create base class for MiniCluster and ExternalMiniCluster This allows utility classes to inte

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 9: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/9/build-support/lint.

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/9/build-support/lint.sh File build-s

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-09 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/9/build-support/lint.sh File build-s

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-09 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/9/build-support/lint.sh File build-

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 9: Build Started http://104.196.14.100/job/kudu-gerrit/3299/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3974 to look at the new patch set (#9). Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Cre

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/6/src/kudu/integration-tests/externa

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 8: Build Started http://104.196.14.100/job/kudu-gerrit/3294/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/3974/6/src/kudu/integration-tests/extern

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/3974/6/src/kudu/integration-tests/extern

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 7: Build Started http://104.196.14.100/job/kudu-gerrit/3289/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-08 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3974 to look at the new patch set (#7). Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Cre

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/3974/6/src/kudu/integration-tests/mini_c

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-01 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: (2 comments) http://gerrit.cloudera.org:8080/#/c/3974/6/src/kudu/integration-tests/exter

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-01 Thread Mike Percy (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3974 to look at the new patch set (#6). Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Cre

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-01 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 6: Build Started http://104.196.14.100/job/kudu-gerrit/3201/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-09-01 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/3974/5/src/kudu/integration-tests/mini_c

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/3974/5/src/kudu/integration-tests/mini_

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-18 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 5: (1 comment) LGTM, one nit below which you can ignore. http://gerrit.cloudera.org:8080/#

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 5: Build Started http://104.196.14.100/job/kudu-gerrit/2905/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-15 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2900/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2884/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2875/ -- To view, visit http://ger

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-14 Thread Mike Percy (Code Review)
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3974 to review the following change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Create ba

[kudu-CR] Create base class for MiniCluster and ExternalMiniCluster

2016-08-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Create base class for MiniCluster and ExternalMiniCluster .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2872/ -- To view, visit http://ger