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 index afd30c1cc321..ef94fe159176 100644 --- 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/presto-product-tests/src/main/resources/com/facebook/presto/tests/jdbc/get_nation_columns.result b/presto-product-tests/src/main/resources/com/facebook/presto/tests/jdbc/get_nation_columns.result index 4ac132a2ccf2..6b274e214a76 100644 --- a/presto-product-tests/src/main/resources/com/facebook/presto/tests/jdbc/get_nation_columns.result +++ b/presto-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|