Andrew Wong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13776 )

Change subject: IMPALA-8727: [DOCS] Impala-side changes for Kudu HMS integration
......................................................................


Patch Set 9: Code-Review+1

(2 comments)

One minor point, otherwise LGTM!

http://gerrit.cloudera.org:8080/#/c/13776/4/docs/shared/impala_common.xml
File docs/shared/impala_common.xml:

http://gerrit.cloudera.org:8080/#/c/13776/4/docs/shared/impala_common.xml@2115
PS4, Line 2115: Kudu service is integrated with the Hive Metastore, the above
              :         operations are not supported.</p>
              :
> Alright, figured it out - Impala's test infra disables validation in the Ku
Ah, nice find!


http://gerrit.cloudera.org:8080/#/c/13776/9/docs/topics/impala_tables.xml
File docs/topics/impala_tables.xml:

http://gerrit.cloudera.org:8080/#/c/13776/9/docs/topics/impala_tables.xml@289
PS9, Line 289: data
It'd be slightly more correct to say "metadata" here



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ieec79ac0bbb860c6394a3bf0617b285a7d23ca9e
Gerrit-Change-Number: 13776
Gerrit-PatchSet: 9
Gerrit-Owner: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Thomas Tauber-Marshall <[email protected]>
Gerrit-Comment-Date: Fri, 19 Jul 2019 01:17:43 +0000
Gerrit-HasComments: Yes

Reply via email to