Skip to content

Commit

Permalink
DAT-18263: review changes - alterViewProperties change priority reduc…
Browse files Browse the repository at this point in the history
…ed to PRIORITY_DATABASE, removed unnecesary dependency on jdbc jar.
  • Loading branch information
Mykhailo Savchenko committed Sep 27, 2024
1 parent d2ec7b3 commit 752533b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
import java.util.ArrayList;
import java.util.List;

import static com.databricks.client.jdbc42.internal.apache.commons.lang.StringUtils.capitalize;
import static liquibase.statement.SqlStatement.EMPTY_SQL_STATEMENT;

@Setter
Expand Down Expand Up @@ -44,7 +43,7 @@ public ValidationErrors validate(Database database) {
protected abstract String getNoPropertiesErrorMessage();

protected String applySubjectToErrorPattern(String subject) {
return MessageFormat.format("Alter {0} Properties change require 'setExtendedTableProperties' or 'unsetExtendedTableProperties' element, please add at least one option.", capitalize(subject));
return MessageFormat.format("Alter {0} Properties change require 'setExtendedTableProperties' or 'unsetExtendedTableProperties' element, please add at least one option.", subject);
}

protected String getConfirmationMessage(String elementName) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
import lombok.Setter;

@Setter
@DatabaseChange(name = "alterViewProperties", description = "Alter View Properties", priority = PrioritizedService.PRIORITY_DATABASE + 500)
@DatabaseChange(name = "alterViewProperties", description = "Alter View Properties", priority = PrioritizedService.PRIORITY_DATABASE)
public class AlterViewPropertiesChangeDatabricks extends AbstractAlterPropertiesChangeDatabricks {

private static final String CHANGE_TYPE_SUBJECT = "View";
Expand Down

0 comments on commit 752533b

Please sign in to comment.