[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has submitted this change and it was merged. Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discuss

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 15: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/5649 To unsubscribe, visit http:

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread Impala Public Jenkins (Code Review)
Impala Public Jenkins has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 15: Build started: http://jenkins.impala.io:8080/job/gerrit-docs-submit/41/ -- To view, visit ht

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 15: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5649 To unsubscribe, visit http://ge

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 14: (2 comments) Just closing off a couple of comments added by me in the last patch set. http://gerrit.c

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 15: (13 comments) http://gerrit.cloudera.org:8080/#/c/5649/14/docs/topics/impala_explain.xml File docs/top

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-16 Thread John Russell (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5649 to look at the new patch set (#15). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Ma

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-06 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 14: (11 comments) http://gerrit.cloudera.org:8080/#/c/5649/14/docs/topics/impala_explain.xml File docs/t

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-06 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 14: OK, maybe "any second" =~ "about a week". But I think all Dimitris's final comments are now addressed.

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-02-06 Thread John Russell (Code Review)
Hello Dimitris Tsirogiannis, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5649 to look at the new patch set (#14). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Ma

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-31 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 13: (8 comments) New patch set coming any second. http://gerrit.cloudera.org:8080/#/c/5649/13/docs/topics

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-31 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 13: John, when do you plan to post a new patch that addresses the last comments? I think after th

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-30 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 13: Code-Review+1 (5 comments) I'll let MJ and/or Todd make a final pass. http://gerrit.cloudera

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-30 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#13). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-30 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 7: (18 comments) Addressed all comments, esp. splitting CREATE TABLE syntax for HDFS vs. Kudu. http://ger

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-26 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#12). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-26 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/5649/11/docs/topics/impala_create_table.xml File doc

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-26 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 11: (1 comment) John, sending a new patch without indicating which comments have been addressed

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-26 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#11). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-25 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/5649/10/docs/topics/impala_kudu.xml File docs/topics

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#10). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 7: (25 comments) http://gerrit.cloudera.org:8080/#/c/5649/7/docs/shared/impala_common.xml File do

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#9). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 8: (15 comments) Addressed some early comments from Dimitris and Todd that I skipped over before. (Some h

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#8). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-24 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 4: (32 comments) Did all of the recent comments. I see some earlier ones still to do. http://gerrit.cloud

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-23 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#7). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-20 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5649/6/docs/shared/impala_common.xml File docs

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-20 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 6: (18 comments) http://gerrit.cloudera.org:8080/#/c/5649/6/docs/shared/impala_common.xml File docs/shar

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-20 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 6: (15 comments) Another round of comments. I've seen that not all previous comments have been a

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-20 Thread Dimitris Tsirogiannis (Code Review)
Dimitris Tsirogiannis has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 6: John, do we have a pdf from these sources? It helps immensely the review process of docs --

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-19 Thread John Russell (Code Review)
John Russell has posted comments on this change. Change subject: [DOCS] Major update to Impala + Kudu page .. Patch Set 6: (36 comments) http://gerrit.cloudera.org:8080/#/c/5649/4/docs/topics/impala_invalidate_metadata.xml Fil

[Impala-ASF-CR] [DOCS] Major update to Impala + Kudu page

2017-01-19 Thread John Russell (Code Review)
John Russell has uploaded a new patch set (#6). Change subject: [DOCS] Major update to Impala + Kudu page .. [DOCS] Major update to Impala + Kudu page Upgrade with details of latest syntax. Fine-tune discussion of PK and other