Update String Type to remove unnecessary conditional and fix createTa…

…ble advanced expected unit test
liquibase · Nov 22, 2023 · d59b891 · d59b891
1 parent e60c5c4
commit d59b891
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 14 deletions.
diff --git a/src/main/java/liquibase/ext/databricks/datatype/StringDatatypeDatabricks.java b/src/main/java/liquibase/ext/databricks/datatype/StringDatatypeDatabricks.java
@@ -27,19 +27,10 @@ public boolean supports(Database database) {
     public DatabaseDataType toDatabaseDataType(Database database) {
         if (database instanceof DatabricksDatabase) {
 
-            DatabaseDataType type = new DatabaseDataType("STRING", this.getParameters());
-            if (this.getParameters().length == 0) {
-                type.setType("STRING");
-            } else {
-                String firstParameter = String.valueOf(this.getParameters()[0]);
-                int stringSize = Integer.parseInt(firstParameter);
-                if (stringSize == 65535) {
-                    type.setType("STRING");
-                }
-                else {
-                    type.setType("STRING");
-                }
-            }
+            DatabaseDataType type = new DatabaseDataType("STRING");
+
+            type.setType("STRING");
+
             return type;
         } else {
             return super.toDatabaseDataType(database);

diff --git a/...esources/liquibase/harness/generateChangelog/expectedChangeLog/databricks/createTable.sql b/...esources/liquibase/harness/generateChangelog/expectedChangeLog/databricks/createTable.sql
@@ -1,3 +1,3 @@
 -- liquibase formatted sql
 
-CREATE TABLE test_table_xml (test_column INT(10));
+CREATE TABLE test_table_xml (test_column INT);