From 28e8b71e7a516bd98d23a0b12164881259b76be3 Mon Sep 17 00:00:00 2001 From: Ashhar Hasan Date: Tue, 2 Nov 2021 13:29:41 +0530 Subject: [PATCH] Move comment above relevant code --- .../src/main/java/io/trino/plugin/jdbc/BaseJdbcClient.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/BaseJdbcClient.java b/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/BaseJdbcClient.java index 52f660f7e399..eb1c3b5ada2c 100644 --- a/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/BaseJdbcClient.java +++ b/plugin/trino-base-jdbc/src/main/java/io/trino/plugin/jdbc/BaseJdbcClient.java @@ -284,10 +284,10 @@ public List getColumns(ConnectorSession session, JdbcTableHand Optional.empty()); Optional columnMapping = toColumnMapping(session, connection, typeHandle); log.debug("Mapping data type of '%s' column '%s': %s mapped to %s", schemaTableName, columnName, typeHandle, columnMapping); - // skip unsupported column types boolean nullable = (resultSet.getInt("NULLABLE") != columnNoNulls); // Note: some databases (e.g. SQL Server) do not return column remarks/comment here. Optional comment = Optional.ofNullable(emptyToNull(resultSet.getString("REMARKS"))); + // skip unsupported column types if (columnMapping.isPresent()) { columns.add(JdbcColumnHandle.builder() .setColumnName(columnName)