You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org> on 2019/08/05 02:25:34 UTC

[kudu-CR] Docker image for python kudu client

Sandish Kumar HN has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14000


Change subject: Docker image for python kudu client
......................................................................

Docker image for python kudu client

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 45 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/14000/1
-- 
To view, visit http://gerrit.cloudera.org:8080/14000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 1
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, Grant Henke, 

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

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

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

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................

KUDU-2849 Docker image for python kudu client

A base image that has all the development libraries required
to run Python Kudu Client.

Building Python Kudu Client Image:
Set Env
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 35 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/14000/4
-- 
To view, visit http://gerrit.cloudera.org:8080/14000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 4
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@225
PS2, Line 225: FROM  build AS kudu-python
> only building from "build" base image is working. nothing is having python,
build is way too large of an image to be used for regular use. 

Can you share the issues you are having using the dev image? That has all of the required dependencies installed. Any issue had should be due to missing files that could be copied from the build image or a configuration issue.

One thing to try is adding `pip install -r requirements.txt` at the start of the run clause. You may need to copy requirements.txt.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 2
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Mon, 05 Aug 2019 03:53:11 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 5:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@226
PS5, Line 226: RUN pip install --upgrade pip
Is this required?


http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@228
PS5, Line 228: COPY ./python/requirements.txt /kudu/python/requirements.txt
Move this with the other COPY statements. You can use the same workdir as below since requirements.txt will get removed after it's used.


http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@231
PS5, Line 231: RUN pip install -r requirements.txt
I think this can all be done in one step/layer in the run below. We should also remove the requirements.txt in that step.

It's important because each action taken in the dockerfile creates a new layer that make the image larger. https://docs.docker.com/develop/develop-images/dockerfile_best-practices/#minimize-the-number-of-layers



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 5
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Mon, 05 Aug 2019 12:36:02 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Sandish Kumar HN has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@225
PS2, Line 225: 
> build is way too large of an image to be used for regular use. 
using dev base image now



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 5
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Mon, 05 Aug 2019 04:38:21 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................

KUDU-2849 Docker image for python kudu client

A base image that has all the development libraries required
to run Python Kudu Client.

Building Python Kudu Client Image:
Set Env
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Reviewed-on: http://gerrit.cloudera.org:8080/14000
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <gr...@apache.org>
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 40 insertions(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Grant Henke: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 7
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, Grant Henke, 

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

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

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

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................

KUDU-2849 Docker image for python kudu client

A base image that has all the development libraries required
to run Python Kudu Client.

Building Python Kudu Client Image:
Set Env
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 42 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/14000/5
-- 
To view, visit http://gerrit.cloudera.org:8080/14000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 5
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>

[kudu-CR] Docker image for python kudu client

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: Docker image for python kudu client
......................................................................


Patch Set 2:

(13 comments)

http://gerrit.cloudera.org:8080/#/c/14000/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/14000/2//COMMIT_MSG@7
PS2, Line 7: Docker image for python kudu client
Add KUDU-2849


http://gerrit.cloudera.org:8080/#/c/14000/2//COMMIT_MSG@9
PS2, Line 9: A base image that has all the development libraries required 
Adjust this line to match the description based on my comments in the dockerfile.


http://gerrit.cloudera.org:8080/#/c/14000/2//COMMIT_MSG@14
PS2, Line 14: Set Env 
Nit: trailing space


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@222
PS2, Line 222: # Builds a base image that has all the development libraries required to run Python Kudu Client.
Maybe "Builds a runtime image with the Kudu python client pre-installed."


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@224
PS2, Line 224: ARG BASE_OS
I don't think you need this here.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@225
PS2, Line 225: FROM  build AS kudu-python
The build image is massive. This doesn't work with the runtime image because it's missing the python stuff. Maybe try using the dev image. If that doesn't work or is too large we may need to adjust the bootstrap scripts so we can just bootstrap the python stuff needed.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@227
PS2, Line 227: 
nit: extra line


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@230
PS2, Line 230: ENV LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/lib64
nit: move this above the workdir so it can be cached through changes.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@232
PS2, Line 232: RUN pip install kudu-python-*.tar.gz
Once we install the client can we removed the original `kudu-python-*.tar.gz` to reduce the image size? Ideally this would be done in the same RUN step with &&.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@234
PS2, Line 234: 
nit: extra line


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@242
PS2, Line 242: LABEL org.label-schema.name="Apache Kudu Python Client Base" \
Maybe remove "Base", since this image is intended to be used/run.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@243
PS2, Line 243:       org.label-schema.description="A base image that has all the Kudu Python \
Update this to match the description above.


http://gerrit.cloudera.org:8080/#/c/14000/2/docker/README.adoc
File docker/README.adoc:

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/README.adoc@113
PS2, Line 113: To run Kudu Python client image:
I don't think you need the extra run guidance here given the other images don't have it. It's also covered in "Running an image" above.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 2
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 05 Aug 2019 03:08:08 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Sandish Kumar HN has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 6:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@226
PS5, Line 226: ENV LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/lib64
> Is this required?
it complains lie "You are using pip version 18.1, however version 19.2.1 is available.", but passes the step


http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@228
PS5, Line 228: # Copy the requirements file.
> Move this with the other COPY statements. You can use the same workdir as b
done


http://gerrit.cloudera.org:8080/#/c/14000/5/docker/Dockerfile@231
PS5, Line 231: COPY --from=build /kudu/python/dist/kudu-python-*.tar.gz .
> I think this can all be done in one step/layer in the run below. We should 
done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 6
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Tue, 06 Aug 2019 22:21:24 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, Grant Henke, 

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

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

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

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................

KUDU-2849 Docker image for python kudu client

A base image that has all the development libraries required
to run Python Kudu Client.

Building Python Kudu Client Image:
Set Env
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 40 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/14000/6
-- 
To view, visit http://gerrit.cloudera.org:8080/14000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 6
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Sandish Kumar HN has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile
File docker/Dockerfile:

http://gerrit.cloudera.org:8080/#/c/14000/2/docker/Dockerfile@225
PS2, Line 225: 
> The build image is massive. This doesn't work with the runtime image becaus
only building from "build" base image is working. nothing is having python, tried third-party base, still got few python related issue's



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 4
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Mon, 05 Aug 2019 03:48:16 +0000
Gerrit-HasComments: Yes

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/14000 )

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................


Patch Set 6: Code-Review+2

I verified this by building and running the following test script in a quickstart environment:

   docker run -it --rm --network="docker_default" apache/kudu:kudu-python-latest

   import kudu
   from kudu.client import Partitioning
   from datetime import datetime
 
   client = kudu.connect(['kudu-master-1','kudu-master-2','kudu-master-3'],['7051','7151','7251'])
   client.list_tables()
   table = client.table('random_user')
   scanner = table.scanner()
   result = scanner.open().read_all_tuples()
   print(result[:1])
   exit()


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 6
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Comment-Date: Wed, 07 Aug 2019 23:36:01 +0000
Gerrit-HasComments: No

[kudu-CR] KUDU-2849 Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, Grant Henke, 

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

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

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

Change subject: KUDU-2849 Docker image for python kudu client
......................................................................

KUDU-2849 Docker image for python kudu client

A base image that has all the development libraries required
to run Python Kudu Client.

Building Python Kudu Client Image:
Set Env
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 35 insertions(+), 0 deletions(-)


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 3
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Grant Henke <gr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

[kudu-CR] Docker image for python kudu client

Posted by "Sandish Kumar HN (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins, 

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

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

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

Change subject: Docker image for python kudu client
......................................................................

Docker image for python kudu client

A base image that has all the development libraries required 
to run Python Kudu Client.


Building Python Kudu Client Image:
Set Env 
export TARGETS=kudu-python
./docker/docker-build.sh

Run docker kudu python client:
docker run --name kudu-python -it apache/kudu:kudu-python-latest

Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
---
M docker/Dockerfile
M docker/README.adoc
2 files changed, 45 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/00/14000/2
-- 
To view, visit http://gerrit.cloudera.org:8080/14000
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7cab8ffe114c73752c261f27ebc7a58e4b57a6e
Gerrit-Change-Number: 14000
Gerrit-PatchSet: 2
Gerrit-Owner: Sandish Kumar HN <sa...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)