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 <[email protected]> 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 <[email protected]> Authored: Wed Nov 30 12:37:21 2016 -0800 Committer: Todd Lipcon <[email protected]> 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 */
