Fix issue with identifying column nullability

prestodb · Aug 12, 2024 · 6c8fd1f · 6c8fd1f
1 parent 88ef79d
commit 6c8fd1f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/connector/system/jdbc/ColumnJdbcTable.java b/presto-main/src/main/java/com/facebook/presto/connector/system/jdbc/ColumnJdbcTable.java
@@ -148,14 +148,14 @@ private static void addColumnRows(Builder builder, String catalog, SchemaTableNa
                     0,
                     decimalDigits(column.getType()),
                     numPrecRadix(column.getType()),
-                    DatabaseMetaData.columnNullableUnknown,
+                    column.isNullable() ? DatabaseMetaData.columnNullable : DatabaseMetaData.columnNoNulls,
                     column.getComment(),
                     null,
                     null,
                     null,
                     charOctetLength(column.getType()),
                     ordinalPosition,
-                    "",
+                    column.isNullable() ? "YES" : "NO",
                     null,
                     null,
                     null,

diff --git a/...product-tests/src/main/resources/com/facebook/presto/tests/jdbc/get_nation_columns.result b/...product-tests/src/main/resources/com/facebook/presto/tests/jdbc/get_nation_columns.result
@@ -1,5 +1,5 @@
 -- delimiter: |; ignoreOrder: false; ignoreExcessRows: false;
-hive|default|nation|n_nationkey|-5|bigint|19|0|null|10|2|null|null|null|null|null|1||null|null|null|null|null|null|
-hive|default|nation|n_name|12|varchar(25)|25|0|null|null|2|null|null|null|null|25|2||null|null|null|null|null|null|
-hive|default|nation|n_regionkey|-5|bigint|19|0|null|10|2|null|null|null|null|null|3||null|null|null|null|null|null|
-hive|default|nation|n_comment|12|varchar(152)|152|0|null|null|2|null|null|null|null|152|4||null|null|null|null|null|null|
+hive|default|nation|n_nationkey|-5|bigint|19|0|null|10|1|null|null|null|null|null|1|YES|null|null|null|null|null|null|
+hive|default|nation|n_name|12|varchar(25)|25|0|null|null|1|null|null|null|null|25|2|YES|null|null|null|null|null|null|
+hive|default|nation|n_regionkey|-5|bigint|19|0|null|10|1|null|null|null|null|null|3|YES|null|null|null|null|null|null|
+hive|default|nation|n_comment|12|varchar(152)|152|0|null|null|1|null|null|null|null|152|4|YES|null|null|null|null|null|null|