Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12534 )

Change subject: [docs] Fix table name in Impala docs
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/12534/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

http://gerrit.cloudera.org:8080/#/c/12534/1/docs/kudu_impala_integration.adoc@231
PS1, Line 231: impala::database_name.table_name
Maybe, it makes sense to mention under which name the table is 
visible/addressable in Kudu and Impala.  What do you think?

The specified  `impala::database_name.table_name` is how the table can be 
addressed in Kudu, right?  What about Impala itself?  That's 
`database_name.table_name` right?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I39866cfc810731e26592ddffd9a328bc8380f46e
Gerrit-Change-Number: 12534
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alex Rodoni <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Wed, 20 Feb 2019 15:31:41 +0000
Gerrit-HasComments: Yes

Reply via email to