You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2016/10/07 18:26:36 UTC

[3/3] kudu git commit: [docs] - Impala doc - partitioning required

[docs] - Impala doc - partitioning required

It seems a blurb in the impala page got missed when partitioning
became required on table create.  Updated verbiage without removing
reference links.

Change-Id: I022b49c4e0b17031b9f14648ef0216ba0094b92b
Reviewed-on: http://gerrit.cloudera.org:8080/4595
Tested-by: Kudu Jenkins
Reviewed-by: Mike Percy <mp...@apache.org>
Reviewed-on: http://gerrit.cloudera.org:8080/4664
Reviewed-by: Dan Burkert <da...@cloudera.com>
Tested-by: Dan Burkert <da...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/5cbcffa5
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/5cbcffa5
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/5cbcffa5

Branch: refs/heads/branch-1.0.x
Commit: 5cbcffa59665e254fb449b51c2d9f0913e882a4f
Parents: 7c5e5c5
Author: Jordan Birdsell <jo...@gmail.com>
Authored: Mon Oct 3 09:23:09 2016 -0400
Committer: Dan Burkert <da...@cloudera.com>
Committed: Fri Oct 7 18:26:06 2016 +0000

----------------------------------------------------------------------
 docs/kudu_impala_integration.adoc | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/5cbcffa5/docs/kudu_impala_integration.adoc
----------------------------------------------------------------------
diff --git a/docs/kudu_impala_integration.adoc b/docs/kudu_impala_integration.adoc
index 706b7b1..971761c 100755
--- a/docs/kudu_impala_integration.adoc
+++ b/docs/kudu_impala_integration.adoc
@@ -365,13 +365,6 @@ to an Impala table, except that you need to write the `CREATE` statement yoursel
 Use the following example as a guideline. Impala first creates the table, then creates
 the mapping.
 
-IMPORTANT: When creating a new Kudu table, you are strongly encouraged to specify
-a distribution scheme. If you do not, your table will consist of a single tablet,
-and thus load will not be distributed across your cluster. See
-<<partitioning_tables>>. The table creation example below is distributed into 16
-buckets by hashing the `id` column, for simplicity. See
-<<partitioning_rules_of_thumb>> for guidelines on partitioning.
-
 [source,sql]
 ----
 CREATE TABLE my_first_table
@@ -401,6 +394,11 @@ For Kudu tables, this must be `com.cloudera.kudu.hive.KuduStorageHandler`.
 `kudu.key_columns`:: the comma-separated list of primary key columns, whose contents
 should not be nullable.
 
+When creating a new Kudu table, you are required to specify a distribution scheme.
+See <<partitioning_tables>>. The table creation example above is distributed into
+16 buckets by hashing the `id` column, for simplicity. See
+<<partitioning_rules_of_thumb>> for guidelines on partitioning.
+
 ==== `CREATE TABLE AS SELECT`
 You can create a table by querying any other table or tables in Impala, using a `CREATE
 TABLE ... AS SELECT` statement. The following example imports all rows from an existing table