From 1740abe739ceb17ff483d7e0a2a3ec236897e31b Mon Sep 17 00:00:00 2001
From: Lukas Eder <lukas.eder@gmail.com>
Date: Thu, 29 Feb 2024 08:53:18 +0100
Subject: [PATCH] [jOOQ/jOOQ#11083] DataType should cache null_() and notNull()
 variants

---
 .../java/org/jooq/impl/AbstractDataTypeX.java | 33 ++++++++++---------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java b/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java
index b629071792..36775de7b0 100644
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java
@@ -85,21 +85,24 @@ abstract class AbstractDataTypeX<T> extends AbstractDataType<T> {
 
     @Override
     public /* non-final */ DataType<T> nullability(Nullability n) {
-        return construct(
-            precision0(),
-            scale0(),
-            length0(),
-            n,
-            hidden(),
-            readonly(),
-            generatedAlwaysAsGenerator(),
-            generationOption(),
-            generationLocation(),
-            collation(),
-            characterSet(),
-            !n.nullable() && identity(),
-            defaultValue()
-        );
+        if (n == nullability())
+            return this;
+        else
+            return construct(
+                precision0(),
+                scale0(),
+                length0(),
+                n,
+                hidden(),
+                readonly(),
+                generatedAlwaysAsGenerator(),
+                generationOption(),
+                generationLocation(),
+                collation(),
+                characterSet(),
+                !n.nullable() && identity(),
+                defaultValue()
+            );
     }
 
     @Override