Properly render NOT.

pom.xml

@@ -5,7 +5,7 @@
 
 	<groupId>org.springframework.data</groupId>
 	<artifactId>spring-data-relational-parent</artifactId>
-	<version>3.2.0-SNAPSHOT</version>
+	<version>3.2.0-1653-where-not-SNAPSHOT</version>
 	<packaging>pom</packaging>
 
 	<name>Spring Data Relational Parent</name>

spring-data-jdbc-distribution/pom.xml

@@ -14,7 +14,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-relational-parent</artifactId>
-		<version>3.2.0-SNAPSHOT</version>
+		<version>3.2.0-1653-where-not-SNAPSHOT</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 

spring-data-jdbc/pom.xml

@@ -6,7 +6,7 @@
 	<modelVersion>4.0.0</modelVersion>
 
 	<artifactId>spring-data-jdbc</artifactId>
-	<version>3.2.0-SNAPSHOT</version>
+	<version>3.2.0-1653-where-not-SNAPSHOT</version>
 
 	<name>Spring Data JDBC</name>
 	<description>Spring Data module for JDBC repositories.</description>
@@ -15,7 +15,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-relational-parent</artifactId>
-		<version>3.2.0-SNAPSHOT</version>
+		<version>3.2.0-1653-where-not-SNAPSHOT</version>
 	</parent>
 
 	<properties>

spring-data-r2dbc/pom.xml

@@ -6,7 +6,7 @@
 	<modelVersion>4.0.0</modelVersion>
 
 	<artifactId>spring-data-r2dbc</artifactId>
-	<version>3.2.0-SNAPSHOT</version>
+	<version>3.2.0-1653-where-not-SNAPSHOT</version>
 
 	<name>Spring Data R2DBC</name>
 	<description>Spring Data module for R2DBC</description>
@@ -15,7 +15,7 @@
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-relational-parent</artifactId>
-		<version>3.2.0-SNAPSHOT</version>
+		<version>3.2.0-1653-where-not-SNAPSHOT</version>
 	</parent>
 
 	<properties>

spring-data-relational/pom.xml

@@ -6,15 +6,15 @@
 	<modelVersion>4.0.0</modelVersion>
 
 	<artifactId>spring-data-relational</artifactId>
-	<version>3.2.0-SNAPSHOT</version>
+	<version>3.2.0-1653-where-not-SNAPSHOT</version>
 
 	<name>Spring Data Relational</name>
 	<description>Spring Data Relational support</description>
 
 	<parent>
 		<groupId>org.springframework.data</groupId>
 		<artifactId>spring-data-relational-parent</artifactId>
-		<version>3.2.0-SNAPSHOT</version>
+		<version>3.2.0-1653-where-not-SNAPSHOT</version>
 	</parent>
 
 	<properties>

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/render/ConditionVisitor.java

@@ -15,16 +15,7 @@
  */
 package org.springframework.data.relational.core.sql.render;
 
-import org.springframework.data.relational.core.sql.AndCondition;
-import org.springframework.data.relational.core.sql.Between;
-import org.springframework.data.relational.core.sql.Comparison;
-import org.springframework.data.relational.core.sql.Condition;
-import org.springframework.data.relational.core.sql.ConstantCondition;
-import org.springframework.data.relational.core.sql.In;
-import org.springframework.data.relational.core.sql.IsNull;
-import org.springframework.data.relational.core.sql.Like;
-import org.springframework.data.relational.core.sql.NestedCondition;
-import org.springframework.data.relational.core.sql.OrCondition;
+import org.springframework.data.relational.core.sql.*;
 import org.springframework.lang.Nullable;
 
 /**
@@ -103,6 +94,10 @@ private DelegatingVisitor getDelegation(Condition segment) {
 			return new ConstantConditionVisitor(context, builder::append);
 		}
 
+		if (segment instanceof Not) {
+			return new NotConditionVisitor(context, builder::append);
+		}
+
 		return null;
 	}
 

spring-data-relational/src/main/java/org/springframework/data/relational/core/sql/render/NotConditionVisitor.java

@@ -0,0 +1,72 @@
+/*
+ * Copyright 2019-2023 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.springframework.data.relational.core.sql.render;
+
+import org.springframework.data.relational.core.sql.Condition;
+import org.springframework.data.relational.core.sql.NestedCondition;
+import org.springframework.data.relational.core.sql.Not;
+import org.springframework.data.relational.core.sql.Visitable;
+import org.springframework.lang.Nullable;
+
+/**
+ * Renderer for {@link Not}. Uses a {@link RenderTarget} to call back for render results.
+ *
+ * @author Jens Schauder
+ * @since 3.2
+ */
+class NotConditionVisitor extends TypedSubtreeVisitor<NestedCondition> {
+
+	private final RenderContext context;
+	private final RenderTarget target;
+
+	private @Nullable ConditionVisitor conditionVisitor;
+
+	NotConditionVisitor(RenderContext context, RenderTarget target) {
+
+		this.context = context;
+		this.target = target;
+	}
+
+	@Override
+	Delegation enterNested(Visitable segment) {
+
+		DelegatingVisitor visitor = getDelegation(segment);
+
+		return visitor != null ? Delegation.delegateTo(visitor) : Delegation.retain();
+	}
+
+	@Nullable
+	private DelegatingVisitor getDelegation(Visitable segment) {
+
+		if (segment instanceof Condition) {
+			return conditionVisitor = new ConditionVisitor(context);
+		}
+
+		return null;
+	}
+
+	@Override
+	Delegation leaveNested(Visitable segment) {
+
+		if (conditionVisitor != null) {
+
+			target.onRendered("NOT (" + conditionVisitor.getRenderedPart() + ")");
+			conditionVisitor = null;
+		}
+
+		return super.leaveNested(segment);
+	}
+}

spring-data-relational/src/test/java/org/springframework/data/relational/core/sql/render/SelectRendererUnitTests.java

@@ -635,6 +635,19 @@ void rendersAliasedExpression() {
 				.isEqualTo("SELECT table.name AS alias FROM table");
 	}
 
+	@Test // GH-1653
+	void notOfNested(){
+
+		Table table = SQL.table("atable");
+
+		Select select = StatementBuilder.select(table.asterisk()).from(table). where(
+				Conditions.nest(table.column("id").isEqualTo(Expressions.just("1"))
+						.and(table.column("id").isEqualTo(Expressions.just("2")))).not()).build();
+		String sql = SqlRenderer.toString(select);
+
+		assertThat(sql).isEqualTo("SELECT atable.* FROM atable WHERE NOT (atable.id = 1 AND atable.id = 2)");
+	}
+
 	/**
 	 * Tests the rendering of analytic functions.
 	 */