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 <[email protected]>
Gerrit-Reviewer: Greg Rahn <[email protected]>
Gerrit-Reviewer: John Russell <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-HasComments: Yes

Reply via email to