impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Russell (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] [DOCS] Take out not-production-ready notice from ADLS page
Date Mon, 24 Jul 2017 19:57:18 GMT
John Russell has posted comments on this change.

Change subject: [DOCS] Take out not-production-ready notice from ADLS page
......................................................................


Patch Set 1:

(2 comments)

If the reusable element is never referenced, it's dead code from the perspective of output,
but sure I'll take it out so the commit history is more emphatic on that point.

http://gerrit.cloudera.org:8080/#/c/7487/1/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

PS1, Line 1118:       <note rev="2.9.0 IMPALA-5333" id="adls_production" type="important">
              :         <p>
              :           Currently, the ADLS support in Impala is preliminary and not
              :           fully tested. Do not use Impala with ADLS in a production environment.
              :         </p>
              :       </note>
> Don't we need to remove this as well?
Done


PS1, Line 1118:       <note rev="2.9.0 IMPALA-5333" id="adls_production" type="important">
              :         <p>
              :           Currently, the ADLS support in Impala is preliminary and not
              :           fully tested. Do not use Impala with ADLS in a production environment.
              :         </p>
              :       </note>
> IMO, yes, it should be removed as well.
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I127b7806feca810503ba3dd000a8e972835e715a
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: Greg Rahn <grahn@cloudera.com>
Gerrit-Reviewer: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sailesh@cloudera.com>
Gerrit-HasComments: Yes

Mime
View raw message