John Russell has posted comments on this change.

Change subject: [DOCS] Replace LZO setup instructions with placeholder
......................................................................


Patch Set 4:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/6439/1/docs/topics/impala_txtfile.xml
File docs/topics/impala_txtfile.xml:

PS1, Line 538: 
> Replace this 'cdh' portion of the filename with a generic placeholder.
Done


http://gerrit.cloudera.org:8080/#/c/6439/2/docs/topics/impala_txtfile.xml
File docs/topics/impala_txtfile.xml:

PS2, Line 562: 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
             : 
> IN particular this section and one at the top seem likely to be universal, 
I'm happy either way. With partial instructions left in, Apache Impala users 
might be able to puzzle through the rest of it themselves. I'll put those 
pieces back. It isn't clear to me though exactly how much of the earlier stuff 
is really relevant. (Will Apache Impala users run any apt-get / yum / zypper 
commands at all or would they be building both impala-lzo and hadoop-lzo from 
source?)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idea5a28a73287b6b3a023843ddada88204db7b1b
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: John Russell <[email protected]>
Gerrit-Reviewer: Laurel Hale <[email protected]>
Gerrit-HasComments: Yes

Reply via email to