Fix errors in table creation when using BIGINT, SMALLINT, TINYINT, or VOID types (#1069)

## Description

This PR fixes an issue in the Databricks JDBC driver where
`PreparedStatement.getMetaData()` would throw exceptions for certain
column types. Specifically, it now correctly handles:

- `BIGINT`  
- `SMALLINT`  
- `TINYINT`  
- `VOID`  

Previously, attempting to get metadata for these types would fail with
exceptions such as `IllegalArgumentException` or
`IllegalStateException`, preventing normal JDBC usage.

**Notes:**

- `INTERVAL` columns are not included in this fix because they cannot
currently be used in `CREATE TABLE` statements, so testing and support
are not possible at this time.
- `GEOMETRY` and `GEOGRAPHY` types are still in preview and tables with
these columns could not be created. As a result, they are not yet fully
supported by `getMetaData()`.

This change ensures that `ResultSetMetaData` is correctly returned for
all supported numeric and void types, improving compatibility and
stability for JDBC clients.

**Fixes**: [databricks-jdbc
#1064](#1064)

## Testing

The following testing was performed to validate the fix:

1. **Unit tests**  
- Added tests covering tables with columns of type `BIGINT`, `SMALLINT`,
`TINYINT`, and `VOID`.
- Verified that calling `getMetaData()` does not throw exceptions and
returns the correct metadata.

2. **Manual testing**  
   - Created a table with a `BIGINT` column and retrieved its metadata:

```java
try (Connection conn = driver.connect(url, props)) {
    try (PreparedStatement st = conn.prepareStatement("create or replace table simple_test_2856371 (col bigint)")) {
        st.execute();
    }
    try (PreparedStatement st = conn.prepareStatement("select t.* from simple_test_2856371 as t")) {
        ResultSetMetaData rsmd = st.getMetaData();
        assert rsmd.getColumnCount() == 1;
        assert "BIGINT".equals(rsmd.getColumnTypeName(1));
    }
}
```

3. **Regression checks**

- Confirmed that other supported column types continue to return
metadata correctly.

- Verified that driver behavior remains consistent across different JDBC
operations.

- Local testing

<img width="1598" height="1039" alt="Screenshot 2025-11-08 at 19 33 10"
src="https://github.com/user-attachments/assets/444ef818-de0d-4ed8-b258-3ebd94b2a256"
/>

<img width="1598" height="1039" alt="Screenshot 2025-11-08 at 19 33 02"
src="https://github.com/user-attachments/assets/41a0bcfa-6147-45b5-9757-638b3c2b4fa8"
/>

---------

Signed-off-by: Roman Dryndik <dryndikroman@gmail.com>
Co-authored-by: Samikshya Chand <148681192+samikshya-db@users.noreply.github.com>

Author: MagnetonBora

NEXT_CHANGELOG.md

@@ -9,5 +9,7 @@
 - Minimized OAuth requests by reducing calls in feature flags and telemetry.
 
 ### Fixed
+- Fixed: Errors in table creation when using BIGINT, SMALLINT, TINYINT, or VOID types.
+- Fixed: PreparedStatement.getMetaData() now correctly reports TINYINT columns as Types.TINYINT (java.lang.Byte) instead of Types.SMALLINT (java.lang.Integer).
 ---
 *Note: When making changes, please add your change under the appropriate section with a brief description.* 

src/main/java/com/databricks/jdbc/common/util/DatabricksTypeUtil.java

@@ -42,7 +42,8 @@ public class DatabricksTypeUtil {
   public static final String INT = "INT";
   public static final String BYTE = "BYTE";
   public static final String VOID = "VOID";
-  public static final String SMALLINT = "SHORT";
+  public static final String SHORT = "SHORT";
+  public static final String SMALLINT = "SMALLINT";
   public static final String NULL = "NULL";
   public static final String STRING = "STRING";
   public static final String TINYINT = "TINYINT";
@@ -68,7 +69,11 @@ public class DatabricksTypeUtil {
               ColumnInfoTypeName.FLOAT,
               ColumnInfoTypeName.INT,
               ColumnInfoTypeName.LONG,
-              ColumnInfoTypeName.SHORT));
+              ColumnInfoTypeName.SHORT,
+              ColumnInfoTypeName.SMALLINT,
+              ColumnInfoTypeName.TINYINT,
+              ColumnInfoTypeName.BYTE,
+              ColumnInfoTypeName.BIGINT));
 
   // only used for PreparedStatement
   public static ColumnInfoTypeName getColumnInfoType(String typeName) {
@@ -81,8 +86,10 @@ public static ColumnInfoTypeName getColumnInfoType(String typeName) {
       case DatabricksTypeUtil.TIMESTAMP_NTZ:
         return ColumnInfoTypeName.TIMESTAMP;
       case DatabricksTypeUtil.SMALLINT:
-      case DatabricksTypeUtil.TINYINT:
+      case DatabricksTypeUtil.SHORT:
         return ColumnInfoTypeName.SHORT;
+      case DatabricksTypeUtil.TINYINT:
+        return ColumnInfoTypeName.TINYINT;
       case DatabricksTypeUtil.BYTE:
         return ColumnInfoTypeName.BYTE;
       case DatabricksTypeUtil.INT:
@@ -118,14 +125,19 @@ public static int getColumnType(ColumnInfoTypeName typeName) {
       return Types.OTHER;
     }
     switch (typeName) {
+      case TINYINT:
       case BYTE:
         return Types.TINYINT;
+      case SMALLINT:
       case SHORT:
         return Types.SMALLINT;
       case INT:
         return Types.INTEGER;
       case LONG:
+      case BIGINT:
         return Types.BIGINT;
+      case VOID:
+        return Types.OTHER;
       case FLOAT:
         return Types.FLOAT;
       case DOUBLE:
@@ -169,8 +181,13 @@ public static String getColumnTypeClassName(ColumnInfoTypeName typeName) {
     switch (typeName) {
       case BYTE:
       case SHORT:
+      case SMALLINT:
+        return "java.lang.Short";
       case INT:
         return "java.lang.Integer";
+      case TINYINT:
+        return "java.lang.Byte";
+      case BIGINT:
       case LONG:
         return "java.lang.Long";
       case FLOAT:
@@ -201,6 +218,7 @@ public static String getColumnTypeClassName(ColumnInfoTypeName typeName) {
       case GEOGRAPHY:
         return GEOGRAPHY_CLASS_NAME;
       case NULL:
+      case VOID:
         return "null";
       case MAP:
         return "java.util.Map";

src/main/java/com/databricks/jdbc/model/core/ColumnInfoTypeName.java

@@ -8,6 +8,10 @@
 public enum ColumnInfoTypeName {
   ARRAY,
   BINARY,
+  BIGINT,
+  SMALLINT,
+  TINYINT,
+  VOID,
   BOOLEAN,
   BYTE,
   CHAR,

src/test/java/com/databricks/jdbc/common/util/DatabricksTypeUtilTest.java

@@ -12,6 +12,7 @@
 import java.sql.SQLException;
 import java.sql.Timestamp;
 import java.sql.Types;
+import java.util.Map;
 import java.util.stream.Stream;
 import org.apache.arrow.vector.types.DateUnit;
 import org.apache.arrow.vector.types.FloatingPointPrecision;
@@ -46,6 +47,11 @@ static Stream<Object[]> dataProvider() {
         new Object[] {TTypeId.TIMESTAMP_TYPE, new ArrowType.Timestamp(TimeUnit.MICROSECOND, null)},
         new Object[] {TTypeId.BINARY_TYPE, ArrowType.Binary.INSTANCE},
         new Object[] {TTypeId.NULL_TYPE, ArrowType.Null.INSTANCE},
+        new Object[] {TTypeId.ARRAY_TYPE, ArrowType.Utf8.INSTANCE},
+        new Object[] {TTypeId.MAP_TYPE, ArrowType.Utf8.INSTANCE},
+        new Object[] {TTypeId.STRUCT_TYPE, ArrowType.Utf8.INSTANCE},
+        new Object[] {TTypeId.USER_DEFINED_TYPE, ArrowType.Utf8.INSTANCE},
+        new Object[] {TTypeId.DECIMAL_TYPE, ArrowType.Utf8.INSTANCE},
         new Object[] {TTypeId.DATE_TYPE, new ArrowType.Date(DateUnit.DAY)});
   }
 
@@ -59,47 +65,82 @@ public void testMapToArrowType(TTypeId typeId, ArrowType expectedArrowType) thro
 
   @Test
   void testGetColumnType() {
-    assertEquals(Types.TINYINT, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.BYTE));
-    assertEquals(Types.SMALLINT, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.SHORT));
-    assertEquals(Types.BIGINT, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.LONG));
-    assertEquals(Types.FLOAT, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.FLOAT));
-    assertEquals(Types.DECIMAL, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.DECIMAL));
-    assertEquals(Types.BINARY, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.BINARY));
-    assertEquals(Types.BOOLEAN, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.BOOLEAN));
-    assertEquals(Types.CHAR, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.CHAR));
-    assertEquals(Types.TIMESTAMP, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.TIMESTAMP));
-    assertEquals(Types.DATE, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.DATE));
-    assertEquals(Types.STRUCT, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.STRUCT));
-    assertEquals(Types.ARRAY, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.ARRAY));
-    assertEquals(Types.VARCHAR, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.NULL));
-    assertEquals(
-        Types.OTHER, DatabricksTypeUtil.getColumnType(ColumnInfoTypeName.USER_DEFINED_TYPE));
+    Map<ColumnInfoTypeName, Integer> expectedMappings =
+        Map.ofEntries(
+            Map.entry(ColumnInfoTypeName.BYTE, Types.TINYINT),
+            Map.entry(ColumnInfoTypeName.SHORT, Types.SMALLINT),
+            Map.entry(ColumnInfoTypeName.SMALLINT, Types.SMALLINT),
+            Map.entry(ColumnInfoTypeName.INT, Types.INTEGER),
+            Map.entry(ColumnInfoTypeName.LONG, Types.BIGINT),
+            Map.entry(ColumnInfoTypeName.BIGINT, Types.BIGINT),
+            Map.entry(ColumnInfoTypeName.TINYINT, Types.TINYINT),
+            Map.entry(ColumnInfoTypeName.VOID, Types.OTHER),
+            Map.entry(ColumnInfoTypeName.FLOAT, Types.FLOAT),
+            Map.entry(ColumnInfoTypeName.DOUBLE, Types.DOUBLE),
+            Map.entry(ColumnInfoTypeName.DECIMAL, Types.DECIMAL),
+            Map.entry(ColumnInfoTypeName.BINARY, Types.BINARY),
+            Map.entry(ColumnInfoTypeName.BOOLEAN, Types.BOOLEAN),
+            Map.entry(ColumnInfoTypeName.CHAR, Types.CHAR),
+            Map.entry(ColumnInfoTypeName.STRING, Types.VARCHAR),
+            Map.entry(ColumnInfoTypeName.MAP, Types.VARCHAR),
+            Map.entry(ColumnInfoTypeName.INTERVAL, Types.VARCHAR),
+            Map.entry(ColumnInfoTypeName.NULL, Types.VARCHAR),
+            Map.entry(ColumnInfoTypeName.TIMESTAMP, Types.TIMESTAMP),
+            Map.entry(ColumnInfoTypeName.DATE, Types.DATE),
+            Map.entry(ColumnInfoTypeName.STRUCT, Types.STRUCT),
+            Map.entry(ColumnInfoTypeName.ARRAY, Types.ARRAY),
+            Map.entry(ColumnInfoTypeName.GEOMETRY, Types.OTHER),
+            Map.entry(ColumnInfoTypeName.GEOGRAPHY, Types.OTHER),
+            Map.entry(ColumnInfoTypeName.USER_DEFINED_TYPE, Types.OTHER));
+
+    expectedMappings.forEach(
+        (typeName, expectedSqlType) ->
+            assertEquals(
+                expectedSqlType,
+                DatabricksTypeUtil.getColumnType(typeName),
+                () -> "Unexpected type for " + typeName));
+    assertEquals(Types.OTHER, DatabricksTypeUtil.getColumnType(null));
   }
 
   @Test
   void testGetColumnTypeClassName() {
-    assertEquals(
-        "java.lang.Long", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.LONG));
-    assertEquals(
-        "java.lang.Float", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.FLOAT));
-    assertEquals(
-        "java.lang.Double", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.DOUBLE));
-    assertEquals(
-        "java.math.BigDecimal",
-        DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.DECIMAL));
-    assertEquals("[B", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.BINARY));
-    assertEquals(
-        "java.sql.Date", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.DATE));
-    assertEquals(
-        "java.sql.Struct", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.STRUCT));
-    assertEquals(
-        "java.sql.Array", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.ARRAY));
-    assertEquals(
-        "java.util.Map", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.MAP));
-    assertEquals("null", DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.NULL));
-    assertEquals(
-        "java.sql.Timestamp",
-        DatabricksTypeUtil.getColumnTypeClassName(ColumnInfoTypeName.TIMESTAMP));
+    final String GEOMETRY_CLASS_NAME = "com.databricks.jdbc.api.IGeometry";
+    final String GEOGRAPHY_CLASS_NAME = "com.databricks.jdbc.api.IGeography";
+    Map<ColumnInfoTypeName, String> expectedMappings =
+        Map.ofEntries(
+            Map.entry(ColumnInfoTypeName.BYTE, "java.lang.Short"),
+            Map.entry(ColumnInfoTypeName.SHORT, "java.lang.Short"),
+            Map.entry(ColumnInfoTypeName.SMALLINT, "java.lang.Short"),
+            Map.entry(ColumnInfoTypeName.INT, "java.lang.Integer"),
+            Map.entry(ColumnInfoTypeName.TINYINT, "java.lang.Byte"),
+            Map.entry(ColumnInfoTypeName.LONG, "java.lang.Long"),
+            Map.entry(ColumnInfoTypeName.BIGINT, "java.lang.Long"),
+            Map.entry(ColumnInfoTypeName.FLOAT, "java.lang.Float"),
+            Map.entry(ColumnInfoTypeName.DOUBLE, "java.lang.Double"),
+            Map.entry(ColumnInfoTypeName.DECIMAL, "java.math.BigDecimal"),
+            Map.entry(ColumnInfoTypeName.BINARY, "[B"),
+            Map.entry(ColumnInfoTypeName.BOOLEAN, "java.lang.Boolean"),
+            Map.entry(ColumnInfoTypeName.CHAR, "java.lang.String"),
+            Map.entry(ColumnInfoTypeName.STRING, "java.lang.String"),
+            Map.entry(ColumnInfoTypeName.INTERVAL, "java.lang.String"),
+            Map.entry(ColumnInfoTypeName.USER_DEFINED_TYPE, "java.lang.String"),
+            Map.entry(ColumnInfoTypeName.TIMESTAMP, "java.sql.Timestamp"),
+            Map.entry(ColumnInfoTypeName.DATE, "java.sql.Date"),
+            Map.entry(ColumnInfoTypeName.STRUCT, "java.sql.Struct"),
+            Map.entry(ColumnInfoTypeName.ARRAY, "java.sql.Array"),
+            Map.entry(ColumnInfoTypeName.GEOMETRY, GEOMETRY_CLASS_NAME),
+            Map.entry(ColumnInfoTypeName.GEOGRAPHY, GEOGRAPHY_CLASS_NAME),
+            Map.entry(ColumnInfoTypeName.MAP, "java.util.Map"),
+            Map.entry(ColumnInfoTypeName.NULL, "null"),
+            Map.entry(ColumnInfoTypeName.VOID, "null"));
+
+    expectedMappings.forEach(
+        (columnType, expectedClassName) ->
+            assertEquals(
+                expectedClassName,
+                DatabricksTypeUtil.getColumnTypeClassName(columnType),
+                () -> "Unexpected type for " + columnType));
+    assertEquals("null", DatabricksTypeUtil.getColumnTypeClassName(null));
   }
 
   @Test
@@ -154,61 +195,42 @@ void testIsSigned() {
 
   @Test
   void testGetDatabricksTypeFromSQLType() {
-    assertEquals(
-        DatabricksTypeUtil.INT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.INTEGER));
-    assertEquals(
-        DatabricksTypeUtil.STRING, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.VARCHAR));
-    assertEquals(
-        DatabricksTypeUtil.CHAR, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.CHAR));
-    assertEquals(
-        DatabricksTypeUtil.STRING,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.LONGVARCHAR));
-    assertEquals(
-        DatabricksTypeUtil.STRING, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.NVARCHAR));
-    assertEquals(
-        DatabricksTypeUtil.STRING,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.LONGNVARCHAR));
-    assertEquals(
-        DatabricksTypeUtil.ARRAY, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.ARRAY));
-    assertEquals(
-        DatabricksTypeUtil.LONG, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.BIGINT));
-    assertEquals(
-        DatabricksTypeUtil.BINARY, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.BINARY));
-    assertEquals(
-        DatabricksTypeUtil.BINARY,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.VARBINARY));
-    assertEquals(
-        DatabricksTypeUtil.BINARY,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.LONGVARBINARY));
-    assertEquals(
-        DatabricksTypeUtil.DECIMAL, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.NUMERIC));
-    assertEquals(
-        DatabricksTypeUtil.DATE, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.DATE));
-    assertEquals(
-        DatabricksTypeUtil.DECIMAL, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.DECIMAL));
-    assertEquals(
-        DatabricksTypeUtil.BOOLEAN, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.BOOLEAN));
-    assertEquals(
-        DatabricksTypeUtil.DOUBLE, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.DOUBLE));
-    assertEquals(
-        DatabricksTypeUtil.FLOAT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.FLOAT));
-    assertEquals(
-        DatabricksTypeUtil.FLOAT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.REAL));
-    assertEquals(
-        DatabricksTypeUtil.TIMESTAMP_NTZ,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.TIMESTAMP));
-    assertEquals(
-        DatabricksTypeUtil.TIMESTAMP,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.TIMESTAMP_WITH_TIMEZONE));
-    assertEquals(
-        DatabricksTypeUtil.STRUCT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.STRUCT));
-    assertEquals(
-        DatabricksTypeUtil.STRUCT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.STRUCT));
-    assertEquals(
-        DatabricksTypeUtil.SMALLINT,
-        DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.SMALLINT));
-    assertEquals(
-        DatabricksTypeUtil.TINYINT, DatabricksTypeUtil.getDatabricksTypeFromSQLType(Types.TINYINT));
+    final int UNKNOWN_TYPE = 1000;
+    final String NULL = "NULL";
+    Map<Integer, String> expectedMappings =
+        Map.ofEntries(
+            Map.entry(Types.INTEGER, DatabricksTypeUtil.INT),
+            Map.entry(Types.VARCHAR, DatabricksTypeUtil.STRING),
+            Map.entry(Types.CHAR, DatabricksTypeUtil.CHAR),
+            Map.entry(Types.LONGVARCHAR, DatabricksTypeUtil.STRING),
+            Map.entry(Types.NVARCHAR, DatabricksTypeUtil.STRING),
+            Map.entry(Types.LONGNVARCHAR, DatabricksTypeUtil.STRING),
+            Map.entry(Types.ARRAY, DatabricksTypeUtil.ARRAY),
+            Map.entry(Types.BIGINT, DatabricksTypeUtil.LONG),
+            Map.entry(Types.BINARY, DatabricksTypeUtil.BINARY),
+            Map.entry(Types.VARBINARY, DatabricksTypeUtil.BINARY),
+            Map.entry(Types.LONGVARBINARY, DatabricksTypeUtil.BINARY),
+            Map.entry(Types.NUMERIC, DatabricksTypeUtil.DECIMAL),
+            Map.entry(Types.DATE, DatabricksTypeUtil.DATE),
+            Map.entry(Types.DECIMAL, DatabricksTypeUtil.DECIMAL),
+            Map.entry(Types.BOOLEAN, DatabricksTypeUtil.BOOLEAN),
+            Map.entry(Types.DOUBLE, DatabricksTypeUtil.DOUBLE),
+            Map.entry(Types.FLOAT, DatabricksTypeUtil.FLOAT),
+            Map.entry(Types.REAL, DatabricksTypeUtil.FLOAT),
+            Map.entry(Types.TIMESTAMP, DatabricksTypeUtil.TIMESTAMP_NTZ),
+            Map.entry(Types.TIMESTAMP_WITH_TIMEZONE, DatabricksTypeUtil.TIMESTAMP),
+            Map.entry(Types.STRUCT, DatabricksTypeUtil.STRUCT),
+            Map.entry(Types.SMALLINT, DatabricksTypeUtil.SMALLINT),
+            Map.entry(Types.TINYINT, DatabricksTypeUtil.TINYINT),
+            Map.entry(Types.BIT, DatabricksTypeUtil.BOOLEAN));
+
+    expectedMappings.forEach(
+        (sqlType, expectedType) ->
+            assertEquals(
+                expectedType,
+                DatabricksTypeUtil.getDatabricksTypeFromSQLType(sqlType),
+                () -> "Unexpected type for " + sqlType));
+    assertEquals(NULL, DatabricksTypeUtil.getDatabricksTypeFromSQLType(UNKNOWN_TYPE));
   }
 
   @Test
@@ -236,7 +258,8 @@ void testInferDatabricksType() {
     "TIMESTAMP, TIMESTAMP",
     "TIMESTAMP_NTZ, TIMESTAMP",
     "SHORT, SHORT",
-    "TINYINT, SHORT",
+    "SMALLINT, SHORT",
+    "TINYINT, TINYINT",
     "BYTE, BYTE",
     "INT, INT",
     "BIGINT, LONG",