Laurel Hale has posted comments on this change.

Change subject: Add Impala 2.9 docs from master branch, with commit hash 
f1a3d8e14dae4948ce77e2f85e036d83f2d8b246
......................................................................


Patch Set 1:

(1 comment)

> (1 comment)

http://gerrit.cloudera.org:8080/#/c/7397/1/impala-docs.html
File impala-docs.html:

PS1, Line 149: impala.pdf
> OK. My motivation for this is probably of lesser importance: it bugs me the
When I open the PDF (impala.pdf), there is no identification on the PDF that 
identifies it as documentation for 2.9 on the cover or in the intro parts of 
the guide. If you change the file name from "impala.pdf" to "impala-2.9.pdf" 
users will be able to at least identify the version by the file name. Since 
people frequently download PDFs to their hard drives, I think it makes sense to 
identify the version in the filename.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id2209088d0120a7b1243113466b33159a4f8a25e
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: asf-site
Gerrit-Owner: Ambreen Kazi <ambreen.k...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org>
Gerrit-Reviewer: John Russell <jruss...@cloudera.com>
Gerrit-Reviewer: Laurel Hale <lau...@cloudera.com>
Gerrit-Reviewer: Michael Brown <mi...@cloudera.com>
Gerrit-HasComments: Yes

Reply via email to