kaspersorensen commented on a change in pull request #241:
URL: https://github.com/apache/metamodel/pull/241#discussion_r451049471



##########
File path: jdbc/src/main/java/org/apache/metamodel/jdbc/JdbcMetadataLoader.java
##########
@@ -383,6 +343,52 @@ private void loadColumns(JdbcTable table, DatabaseMetaData 
metaData) {
         }
     }
 
+    private void processColumn(JdbcTable table, boolean convertLobs, ResultSet 
rs, int columnNumber) throws SQLException {
+        final String columnName = rs.getString(4);
+        if (_identifierQuoteString == null && new 
StringTokenizer(columnName).countTokens() > 1) {
+            logger.warn("column name contains whitespace: \"" + columnName + 
"\".");
+        }
+
+        final int jdbcType = rs.getInt(5);
+        final String nativeType = rs.getString(6);
+        final Integer columnSize = rs.getInt(7);
+        Integer decimalDigits = rs.getInt(9);

Review comment:
       Doesn't look like it's fixed. Did you push the change?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to