Todd Lipcon has posted comments on this change.

Change subject: [doc] KUDU-1630 impala_tables.html link is showing as text and 
not actual link
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4483/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

Line 349: See link: 
http://www.cloudera.com/content/cloudera/en/documentation/core/latest/topics/impala_tables.html
I think it would be preferable to give this link some actual text. like "See 
the Impala documentation for more information about internal and external 
tables." (with a link from 'Impala documentation' to the appropriate page).


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I25c1251b312586b38a558eb3365f60f7ff3a7ba0
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Ninad Shringarpure <ni...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to