Alex Rodoni has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10080 )

Change subject: IMPALA-6860: [DOCS] Upgrade considerations for Impala 3.0
......................................................................


Patch Set 3:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml
File docs/topics/impala_upgrading.xml:

http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@150
PS3, Line 150:           identifier, e.g. a table name, enclose the word in 
quotes.
> enclose the word in backticks.
Done


http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@155
PS3, Line 155:           startup flag: 
<codeph>--reserved_words_version=2.11.0</codeph>.
> Note that this startup option will eventually be deprecated.
Done


http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@184
PS3, Line 184:           To conform to the SQL standard, Impala no longer 
performs alias
> probably better to just link to the documentation and not repeat all exampl
Done


http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@190
PS3, Line 190:           For example, the following statements will result in 
syntax errors.
> They will not result in syntax errors, but error in semantic analysis (prob
Done


http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@234
PS3, Line 234:           The <codeph>PARQUET_ARRAY_RESOLUTION</codeph> query 
option controls
> Remove this paragraph and merge the last sentence linking to the query opti
Done


http://gerrit.cloudera.org:8080/#/c/10080/3/docs/topics/impala_upgrading.xml@247
PS3, Line 247:             >clustered</keyword> hint is enabled by default. The 
hint inserts a
> The hint adds a local sort...
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If8416ac0abb7ea1b918ba53b9533af27182fbe89
Gerrit-Change-Number: 10080
Gerrit-PatchSet: 3
Gerrit-Owner: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com>
Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Vuk Ercegovac <vercego...@cloudera.com>
Gerrit-Comment-Date: Thu, 19 Apr 2018 22:33:12 +0000
Gerrit-HasComments: Yes

Reply via email to