From 3bd5d16ad06b072fd9014f2bbeebd80a6affb110 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Wed, 25 Feb 2015 18:18:08 +0100 Subject: [PATCH] [#3727] Don't apply this fix to databases other than PostgreSQL --- jOOQ/src/main/java/org/jooq/impl/Function.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/Function.java b/jOOQ/src/main/java/org/jooq/impl/Function.java index 1d7deaeb86..f89433b453 100644 --- a/jOOQ/src/main/java/org/jooq/impl/Function.java +++ b/jOOQ/src/main/java/org/jooq/impl/Function.java @@ -370,7 +370,10 @@ class Function extends AbstractField implements // [#3727] Referenced WindowDefinitions that contain a frame clause // shouldn't be referenced from within parentheses (in PostgreSQL) if (windowDefinition != null) - return windowDefinition; + if (POSTGRES == ctx.family()) + return windowDefinition; + else + return DSL.sql("{0}", windowDefinition); // [#531] Inline window specifications if the WINDOW clause is not supported if (windowName != null) { @@ -384,7 +387,7 @@ class Function extends AbstractField implements if (windows != null) { for (WindowDefinition window : windows) { if (((WindowDefinitionImpl) window).getName().equals(windowName)) { - return window; + return DSL.sql("({0})", window); } } }