Laurel Hale has posted comments on this change.

Change subject: Patch references to Cloudera and CDH in Impala tutorial
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

Change the article before the <hadoop_distro> keydef and you're golden. The 
value that replaces <hadoop_distro> is "Apache Hadoop" and therefore requires 
the article "an" instead of "a."

http://gerrit.cloudera.org:8080/#/c/5663/2/docs/topics/impala_tutorial.xml
File docs/topics/impala_tutorial.xml:

PS2, Line 60: a
This should be "an" since the value that replaces "hadoop_distro" begins with 
the word, "Apache," a vowel. Sorry. :-(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I44245b65ce6f247ae8771f582f4b33c3712145ae
Gerrit-PatchSet: 2
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