added clob as string alias

liquibase · Jul 30, 2024 · e4b37ba · e4b37ba
1 parent 906c6e7
commit e4b37ba
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/src/main/java/liquibase/ext/databricks/database/DatabricksDatabase.java b/src/main/java/liquibase/ext/databricks/database/DatabricksDatabase.java
@@ -30,7 +30,7 @@ public class DatabricksDatabase extends AbstractJdbcDatabase {
     private Set<String> systemTablesAndViews = new HashSet<>();
 
     //Define data type names enabled for auto-increment columns - currently only BIGINT
-    public static final List<String> VALID_AUTO_INCREMENT_COLUMN_TYPE_NAMES = Collections.unmodifiableList(Arrays.asList("BIGINT"));
+    public static final List<String> VALID_AUTO_INCREMENT_COLUMN_TYPE_NAMES = Collections.singletonList("BIGINT");
 
     public DatabricksDatabase() {
         super.setCurrentDateTimeFunction("current_timestamp()");
@@ -133,6 +133,11 @@ public boolean supportsTablespaces() {
     @Override
     public boolean supportsSequences() { return false; }
 
+    @Override
+    public boolean supportsDatabaseChangeLogHistory() {
+        return true;
+    }
+
     @Override
     public String getAutoIncrementClause(final BigInteger startWith, final BigInteger incrementBy, final String generationType, final Boolean defaultOnNull) {
         if (!this.supportsAutoIncrement()) {

diff --git a/src/main/java/liquibase/ext/databricks/datatype/StringDatatypeDatabricks.java b/src/main/java/liquibase/ext/databricks/datatype/StringDatatypeDatabricks.java
@@ -4,23 +4,25 @@
 import liquibase.database.Database;
 import liquibase.datatype.DataTypeInfo;
 import liquibase.datatype.DatabaseDataType;
-import liquibase.datatype.LiquibaseDataType;
+import liquibase.datatype.core.VarcharType;
 import liquibase.ext.databricks.database.DatabricksDatabase;
 
 @DataTypeInfo(
         name = "string",
         minParameters = 0,
         maxParameters = 0,
-        priority = DatabricksDatabase.DATABRICKS_PRIORITY_DATABASE
+        priority = DatabricksDatabase.DATABRICKS_PRIORITY_DATABASE,
+        aliases = { "varchar", "clob", "java.lang.String" }
 )
-public class StringDatatypeDatabricks extends LiquibaseDataType {
+public class StringDatatypeDatabricks extends VarcharType {
     public StringDatatypeDatabricks() {
     }
 
+    @Override
     public boolean supports(Database database) {
         return database instanceof DatabricksDatabase;
     }
-
+    @Override
     public DatabaseDataType toDatabaseDataType(Database database) {
         if (database instanceof DatabricksDatabase) {