impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Russell (Code Review)" <>
Subject [Impala-ASF-CR] Patch references to Cloudera and CDH in Impala tutorial
Date Tue, 31 Jan 2017 22:16:38 GMT
Hello Laurel Hale,

I'd like you to reexamine a change.  Please visit

to look at the new patch set (#2).

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

Patch references to Cloudera and CDH in Impala tutorial

There was one tutorial that actually ran under the
'cloudera' user and so repeated that name over and
over in directory and HDFS paths. I switched that to

I suppressed some <note> and <li> tags with Cloudera
Manager-specific details. Will physically remove those
from the source in a subsequent iteration.

I left several instances of audience="Cloudera" because
those will be changed to audience="hidden" as part of
a separate change request.

I marked with rev="upstream" some <codeblock> tags
containing impala-shell banners with a Cloudera copyright
statement. Will decide on a convention to handle those
(elide those lines, or use a conref to consistently
substitute the generic equivalent) and do that in a
followup patch set.

Change-Id: I44245b65ce6f247ae8771f582f4b33c3712145ae
M docs/impala_keydefs.ditamap
M docs/topics/impala_tutorial.xml
2 files changed, 26 insertions(+), 26 deletions(-)

  git pull ssh:// refs/changes/63/5663/2
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I44245b65ce6f247ae8771f582f4b33c3712145ae
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <>
Gerrit-Reviewer: Jim Apple <>
Gerrit-Reviewer: Laurel Hale <>

View raw message