impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jim Apple (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] Take out (stale) architecture diagram w/ Cloudera-specific terminology.
Date Tue, 03 Jan 2017 23:43:55 GMT
Jim Apple has posted comments on this change.

Change subject: Take out (stale) architecture diagram w/ Cloudera-specific terminology.
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/5596/1/docs/topics/impala_intro.xml
File docs/topics/impala_intro.xml:

Line 73:     <title>How Impala Works with <keyword keyref="hadoop_distro"/></title>
Should this generally be made even more generic for users who don't use a distro?


Line 82:       <p audience="Cloudera" conref="../shared/impala_common.xml#common/impala_overview_diagram"/>
long line


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iab3786f113a5492f3aeac7480bfbfdb030cedf5b
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: Ambreen Kazi <ambreen.kazi@cloudera.com>
Gerrit-Reviewer: Anonymous Coward #250
Gerrit-Reviewer: Jim Apple <jbapple-impala@apache.org>
Gerrit-Reviewer: John Russell <jrussell@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message