From f95bc28ebab47b86bdb7101ff6d8e4ccfb014ed3 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Wed, 3 Jun 2020 16:56:05 +0200 Subject: [PATCH] [jOOQ/jOOQ#681] Fix ALTER DOMAIN .. DROP CONSTRAINT IF EXISTS for PG --- jOOQ/src/main/java/org/jooq/impl/AlterDomainImpl.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/AlterDomainImpl.java b/jOOQ/src/main/java/org/jooq/impl/AlterDomainImpl.java index e9f0a7f309..f54d7cfc02 100644 --- a/jOOQ/src/main/java/org/jooq/impl/AlterDomainImpl.java +++ b/jOOQ/src/main/java/org/jooq/impl/AlterDomainImpl.java @@ -309,7 +309,7 @@ implements private static final Set NO_SUPPORT_RENAME_CONSTRAINT_IF_EXISTS = SQLDialect.supportedBy(FIREBIRD, POSTGRES); - private static final Set NO_SUPPORT_DROP_CONSTRAINT_IF_EXISTS = SQLDialect.supportedBy(FIREBIRD, POSTGRES); + private static final Set NO_SUPPORT_DROP_CONSTRAINT_IF_EXISTS = SQLDialect.supportedBy(FIREBIRD); private final boolean supportsRenameConstraintIfExists(Context ctx) { return !NO_SUPPORT_RENAME_CONSTRAINT_IF_EXISTS.contains(ctx.family()); @@ -321,7 +321,8 @@ implements @Override public final void accept(Context ctx) { - if (renameConstraintIfExists && !supportsRenameConstraintIfExists(ctx)) { + if (renameConstraintIfExists && !supportsRenameConstraintIfExists(ctx) || + dropConstraintIfExists && !supportsDropConstraintIfExists(ctx)) { Tools.beginTryCatch(ctx, DDLStatementType.ALTER_DOMAIN); accept0(ctx); Tools.endTryCatch(ctx, DDLStatementType.ALTER_DOMAIN);