[kudu-CR] KUDU-1331] Add scripts to build environment easier with Docker

2016-06-20 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change.

Change subject: KUDU-1331] Add scripts to build environment easier with Docker
..


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2162/3/start-build-env.sh
File start-build-env.sh:

Not a review, but is it strictly necessary for this file to be in Kudu's 
top-level directory? It's not something that's applicable to every 
user/developer, so it seems like build-support would be a better home.


-- 
To view, visit http://gerrit.cloudera.org:8080/2162
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc110e95fb8f43cfe5ee415b65777a254408be52
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Tsuyoshi Ozawa 
Gerrit-Reviewer: Adar Dembo 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: Yes


[kudu-CR] KUDU-1331] Add scripts to build environment easier with Docker

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change.

Change subject: KUDU-1331] Add scripts to build environment easier with Docker
..


Patch Set 3:

Build Started http://104.196.14.100/job/kudu-gerrit/1873/

-- 
To view, visit http://gerrit.cloudera.org:8080/2162
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc110e95fb8f43cfe5ee415b65777a254408be52
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Tsuyoshi Ozawa 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: No


[kudu-CR] KUDU-1331] Add scripts to build environment easier with Docker

2016-06-20 Thread Tsuyoshi Ozawa (Code Review)
Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

http://gerrit.cloudera.org:8080/2162

to look at the new patch set (#3).

Change subject: KUDU-1331] Add scripts to build environment easier with Docker
..

KUDU-1331] Add scripts to build environment easier with Docker

This script installs all dependencies with Docker.

Change-Id: Idc110e95fb8f43cfe5ee415b65777a254408be52
---
A build-support/tools/docker/Dockerfile
A start-build-env.sh
2 files changed, 81 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/2162/3
-- 
To view, visit http://gerrit.cloudera.org:8080/2162
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idc110e95fb8f43cfe5ee415b65777a254408be52
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Tsuyoshi Ozawa 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 


[kudu-CR] KUDU-1331] Add scripts to build environment easier with Docker

2016-06-20 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change.

Change subject: KUDU-1331] Add scripts to build environment easier with Docker
..


Patch Set 2:

Build Started http://104.196.14.100/job/kudu-gerrit/1871/

-- 
To view, visit http://gerrit.cloudera.org:8080/2162
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc110e95fb8f43cfe5ee415b65777a254408be52
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Tsuyoshi Ozawa 
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon 
Gerrit-HasComments: No