Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15397 )

Change subject: [quickstart] Add an Apache Impala quickstart guide
......................................................................


Patch Set 1:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc
File examples/quickstart/impala/README.adoc:

http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@32
PS1, Line 32: s
> No plural.
Done


http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@38
PS1, Line 38: apache/kudu:impala-latest
> From where will this docker image be pulled from?
The image will be pulled automatically from docker hub if the user doesn't have 
it local. See my commit message about testing before the apache image is 
published.


http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@36
PS1, Line 36: docker run -d --name kudu-impala --network="docker_default" \
            :   -p 21000:21000 -p 21050:21050 -p 25000:25000 -p 25010:25010 -p 
25020:25020 \
            :   apache/kudu:impala-latest impala
> Could you add some explanation about what's being brought up here?
I extended my description above.


http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@60
PS1, Line 60: they
> then
Done


http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@66
PS1, Line 66: crete
> create
Done


http://gerrit.cloudera.org:8080/#/c/15397/1/examples/quickstart/impala/README.adoc@155
PS1, Line 155: ating
> creating?
This section was accidentally left in. I removed it.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I208623add96f47119d36ac975b04352854398983
Gerrit-Change-Number: 15397
Gerrit-PatchSet: 1
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 10 Mar 2020 22:41:00 +0000
Gerrit-HasComments: Yes

Reply via email to