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] IMPALA-5137: [DOCS] Document TIMESTAMP for Kudu tables
Date Fri, 02 Jun 2017 15:11:56 GMT
John Russell has posted comments on this change.

Change subject: IMPALA-5137: [DOCS] Document TIMESTAMP for Kudu tables
......................................................................


Patch Set 2:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/7035/1/docs/topics/impala_datetime_functions.xml
File docs/topics/impala_datetime_functions.xml:

Line 1621:           <p conref="../shared/impala_common.xml#common/kudu_blurb"/>
> tabs?
Huh. Must not have the right tabs-to-spaces editor config on the laptop where I did these
edits.


http://gerrit.cloudera.org:8080/#/c/7035/1/docs/topics/impala_kudu.xml
File docs/topics/impala_kudu.xml:

Line 972:   partition by hash (id) partitions 20,
> tabs?
Done


http://gerrit.cloudera.org:8080/#/c/7035/1/docs/topics/impala_timestamp.xml
File docs/topics/impala_timestamp.xml:

Line 459:   partition by hash (x) partitions 16
> tabs?
Done


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib889198eb2c918c969c7613dd1ddf65a801f7926
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: John Russell <jrussell@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <todd@apache.org>
Gerrit-HasComments: Yes

Mime
View raw message