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