You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/12/01 01:15:05 UTC

[2/9] kudu git commit: KUDU-1754. Add a warning to the Java API docs that columns default NOT NULL

KUDU-1754. Add a warning to the Java API docs that columns default NOT NULL

Change-Id: Ia5589746f8b618874326fce69532578468e8ca4a
Reviewed-on: http://gerrit.cloudera.org:8080/5283
Reviewed-by: Dan Burkert <da...@apache.org>
Tested-by: Kudu Jenkins


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

Branch: refs/heads/master
Commit: 85245160b73055b548f8a0b79be1786212cc6afb
Parents: 885d2e9
Author: Todd Lipcon <to...@apache.org>
Authored: Wed Nov 30 12:37:21 2016 -0800
Committer: Todd Lipcon <to...@apache.org>
Committed: Wed Nov 30 21:21:46 2016 +0000

----------------------------------------------------------------------
 .../src/main/java/org/apache/kudu/ColumnSchema.java           | 7 +++++++
 1 file changed, 7 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/85245160/java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java b/java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java
index e4d93e4..250ab56 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/ColumnSchema.java
@@ -239,6 +239,13 @@ public class ColumnSchema {
 
     /**
      * Marks the column as allowing null values. False by default.
+     * <p>
+     * <strong>NOTE:</strong> the "not-nullable-by-default" behavior here differs from
+     * the equivalent API in the Python and C++ clients. It also differs from the
+     * standard behavior of SQL <code>CREATE TABLE</code> statements. It is
+     * recommended to always specify nullability explicitly using this API
+     * in order to avoid confusion.
+     *
      * @param nullable a boolean that indicates if the column allows null values
      * @return this instance
      */