You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@cloudera.org> on 2017/01/18 20:10:20 UTC
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Hello Ambreen Kazi,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/5733
to review the following change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Update Impala docs for upcoming Impala 2.8 release
* No longer needs to document the special 'IMPALA_KUDU' build.
* Fixed syntax for new style
* Fixed a few typos/issues elsewhere (eg use of 'int32' instead of 'int'
type)
* Commented out the docs for composite range partitioning, which seems to
no longer be supported in Impala.
Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
---
M docs/kudu_impala_integration.adoc
1 file changed, 138 insertions(+), 346 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/5733/1
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Ambreen Kazi (Code Review)" <ge...@cloudera.org>.
Ambreen Kazi has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5733/2/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS2, Line 158: Tables are divided into tablets which are each served by one or more tablet
: servers. Ideally, tablets should split a table's data relatively equally. Kudu currently
: has no mechanism for automatically (or manually) splitting a pre-existing tablet.
: Until this feature has been implemented, *you must specify your partitioning when
: creating a table*. When designing your table schema, consider primary keys that will allow you to
: split your table into partitions which grow at similar rates. You can designate
: partitions using a `PARTITION BY` clause when creating a table using Impala:
:
: NOTE: Impala keywords, such as `group`, are enclosed by back-tick characters when
: they are not used in their keyword sense.
This text has been duplicated in the Partitioning Tables section below.
PS2, Line 370: `DISTRIBUTE
: BY HASH (id, sku)`.
old syntax - DISTRIBUTE BY
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for Impala 2.8 release
Posted by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org>.
Jean-Daniel Cryans has posted comments on this change.
Change subject: Update Impala docs for Impala 2.8 release
......................................................................
Patch Set 3: Verified+1
Unrelated flake.
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5733
to look at the new patch set (#2).
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Update Impala docs for upcoming Impala 2.8 release
* No longer needs to document the special 'IMPALA_KUDU' build.
* Fixed syntax for new style
* Added back CTAS, which has syntax (we'd mistakenly thought it was
removed)
* Fixed a few typos/issues elsewhere (eg use of 'int32' instead of 'int'
type)
* Removed the docs for composite range partitioning, which seems to
no longer be supported in Impala.
Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
---
M docs/kudu_impala_integration.adoc
M docs/quickstart.adoc
2 files changed, 148 insertions(+), 400 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/5733/2
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 2:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/5733/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS1, Line 39: Impala 2.8
> I think unfortunately the version in the code will still say 2.7, but we're
ugh... yea I just checked on a CDH 5.10 cluster and it still says 5.7. I changed up this section to match.
http://gerrit.cloudera.org:8080/#/c/5733/2/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS2, Line 144: same names and types as the columns in `old_table`
> technically its the names and types of the columns in the select stmt; you
Done
PS2, Line 158: Tables are divided into tablets which are each served by one or more tablet
: servers. Ideally, tablets should split a table's data relatively equally. Kudu currently
: has no mechanism for automatically (or manually) splitting a pre-existing tablet.
: Until this feature has been implemented, *you must specify your partitioning when
: creating a table*. When designing your table schema, consider primary keys that will allow you to
: split your table into partitions which grow at similar rates. You can designate
: partitions using a `PARTITION BY` clause when creating a table using Impala:
:
: NOTE: Impala keywords, such as `group`, are enclosed by back-tick characters when
: they are not used in their keyword sense.
> This text has been duplicated in the Partitioning Tables section below.
Done
PS2, Line 370: `DISTRIBUTE
: BY HASH (id, sku)`.
> old syntax - DISTRIBUTE BY
Done
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for Impala 2.8 release
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5733
to look at the new patch set (#3).
Change subject: Update Impala docs for Impala 2.8 release
......................................................................
Update Impala docs for Impala 2.8 release
* No longer needs to document the special 'IMPALA_KUDU' build.
* Fixed syntax for new style
* Added back CTAS, which has syntax (we'd mistakenly thought it was
removed)
* Fixed a few typos/issues elsewhere (eg use of 'int32' instead of 'int'
type)
* Removed the docs for composite range partitioning, which seems to
no longer be supported in Impala.
Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
---
M docs/kudu_impala_integration.adoc
M docs/quickstart.adoc
2 files changed, 156 insertions(+), 403 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/33/5733/3
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5733/2/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS2, Line 144: same names and types as the columns in `old_table`
technically its the names and types of the columns in the select stmt; you could alias, cast, or join.
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Todd Lipcon (Code Review)" <ge...@cloudera.org>.
Todd Lipcon has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/5733/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
Line 30: Kudu has tight integration with Apache Impala, allowing you to use Impala
> (incubating) necessary?
typically it's OK to use "incubating" only in the first reference in a document, iirc, and it's referenced as incubating in the title. But I'll add it here just in case
PS1, Line 39: Impala 2.8
question: I just noticed that the build I'm testing (CDH 5.10 pre-release) says:
impalad version 2.7.0-cdh5.10.0 RELEASE (build 1c57740759e175d8810c6790af3c5a60862a0ce9
even though it seems to correspond more or less to upstream 2.8. Should I instead be documenting this in a different way? Or will the final bits say 2.8?
PS1, Line 365: // TODO(todd) Composite partition keys for range partitioning are not allowed
: // anymore in Impala 2.8?
> Except when specifying '=', i.e. no n-dimensional ranges
ok, I'm just going to delete this section for now, and assume that this document doesn't need to be fully comprehensive.
PS1, Line 703: regardless of whether the table is an internal or external
: table. This avoids disruption to other applications that may be accessing the
: underlying Kudu table.
> this isn't true, internal tables will be renamed. I was surprised by this a
It's true in the version I'm testing with: https://gist.github.com/a2c5e6a869459a7121f2e71b29c86677
impalad version 2.7.0-cdh5.10.0 RELEASE (build 1c57740759e175d8810c6790af3c5a60862a0ce9
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 1:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5733/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS1, Line 39: Impala 2.8
> question: I just noticed that the build I'm testing (CDH 5.10 pre-release)
I think unfortunately the version in the code will still say 2.7, but we're talking about 2.8. I guess this will be like a known issue. I'll double check.
PS1, Line 703: regardless of whether the table is an internal or external
: table. This avoids disruption to other applications that may be accessing the
: underlying Kudu table.
> It's true in the version I'm testing with: https://gist.github.com/a2c5e6a8
My bad
PS1, Line 707: .Rename the underlying Kudu table for an internal table
:
: If a table is an internal table, the underlying Kudu table may be renamed by
: changing the `kudu.table_name` property:
> I think we wanna talk about this for external tables, given what I said abo
ignore; this is too complicated I can't even keep it straight I have to try it every time :/
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for Impala 2.8 release
Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has posted comments on this change.
Change subject: Update Impala docs for Impala 2.8 release
......................................................................
Patch Set 3: Code-Review+1
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5733/3/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
PS3, Line 41: ** The version of Impala 2.7.0 that ships with CDH 5.10. `SELECT VERSION()` will
: report `impalad version 2.7.0-cdh5.10.0`.
: ** Apache Impala 2.8.0 releases compiled from source. `SELECT VERSION()` will
: report `impalad version 2.8.0`.
sorry :(
thanks
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for upcoming Impala 2.8 release
Posted by "Matthew Jacobs (Code Review)" <ge...@cloudera.org>.
Matthew Jacobs has posted comments on this change.
Change subject: Update Impala docs for upcoming Impala 2.8 release
......................................................................
Patch Set 1:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/5733/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:
Line 30: Kudu has tight integration with Apache Impala, allowing you to use Impala
(incubating) necessary?
PS1, Line 365: // TODO(todd) Composite partition keys for range partitioning are not allowed
: // anymore in Impala 2.8?
Except when specifying '=', i.e. no n-dimensional ranges
... (PARTITION VALUE = (1, 'Martin'))
PS1, Line 703: regardless of whether the table is an internal or external
: table. This avoids disruption to other applications that may be accessing the
: underlying Kudu table.
this isn't true, internal tables will be renamed. I was surprised by this as well but IIWII at this point.
PS1, Line 707: .Rename the underlying Kudu table for an internal table
:
: If a table is an internal table, the underlying Kudu table may be renamed by
: changing the `kudu.table_name` property:
I think we wanna talk about this for external tables, given what I said above
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-HasComments: Yes
[kudu-CR] Update Impala docs for Impala 2.8 release
Posted by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org>.
Jean-Daniel Cryans has submitted this change and it was merged.
Change subject: Update Impala docs for Impala 2.8 release
......................................................................
Update Impala docs for Impala 2.8 release
* No longer needs to document the special 'IMPALA_KUDU' build.
* Fixed syntax for new style
* Added back CTAS, which has syntax (we'd mistakenly thought it was
removed)
* Fixed a few typos/issues elsewhere (eg use of 'int32' instead of 'int'
type)
* Removed the docs for composite range partitioning, which seems to
no longer be supported in Impala.
Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Reviewed-on: http://gerrit.cloudera.org:8080/5733
Reviewed-by: Matthew Jacobs <mj...@cloudera.com>
Reviewed-by: Jean-Daniel Cryans <jd...@apache.org>
Tested-by: Jean-Daniel Cryans <jd...@apache.org>
---
M docs/kudu_impala_integration.adoc
M docs/quickstart.adoc
2 files changed, 156 insertions(+), 403 deletions(-)
Approvals:
Jean-Daniel Cryans: Looks good to me, approved; Verified
Matthew Jacobs: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
[kudu-CR] Update Impala docs for Impala 2.8 release
Posted by "Jean-Daniel Cryans (Code Review)" <ge...@cloudera.org>.
Jean-Daniel Cryans has posted comments on this change.
Change subject: Update Impala docs for Impala 2.8 release
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5733
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia43d18e8d92c52e5868e1d48b91351bca41b53f8
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Ambreen Kazi <am...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jd...@apache.org>
Gerrit-Reviewer: John Russell <jr...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Matthew Jacobs <mj...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No