diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractLazyScope.java b/jOOQ/src/main/java/org/jooq/impl/AbstractLazyScope.java new file mode 100644 index 0000000000..5bec8563b0 --- /dev/null +++ b/jOOQ/src/main/java/org/jooq/impl/AbstractLazyScope.java @@ -0,0 +1,133 @@ +/* + * 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. + * + * Other licenses: + * ----------------------------------------------------------------------------- + * Commercial licenses for this work are available. These replace the above + * ASL 2.0 and offer limited warranties, support, maintenance, and commercial + * database integrations. + * + * For more information, please visit: https://www.jooq.org/legal/licensing + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + */ +package org.jooq.impl; + +import java.time.Instant; +import java.util.Map; + +import org.jooq.CacheContext; +import org.jooq.Configuration; +import org.jooq.ConverterContext; +import org.jooq.DSLContext; +import org.jooq.GeneratorContext; +import org.jooq.SQLDialect; +import org.jooq.Scope; +import org.jooq.conf.Settings; + +/** + * Unlike {@link AbstractScope}, this implementation lazy initialises its + * {@link #data()} map in order to avoid allocation in very short lived + * {@link Scope} implementations, such as e.g. {@link CacheContext}, + * {@link ConverterContext}, {@link GeneratorContext}. + * + * @author Lukas Eder + */ +abstract class AbstractLazyScope implements Scope { + + private final Configuration configuration; + private Map data; + private final Instant creationTime; + + AbstractLazyScope(Configuration configuration) { + this(configuration, null); + } + + AbstractLazyScope(Configuration configuration, Map data) { + + // The Configuration can be null when unattached objects are + // executed or when unattached Records are stored... + if (configuration == null) + configuration = new DefaultConfiguration(); + + this.configuration = configuration; + this.data = data; + this.creationTime = configuration.clock().instant(); + } + + // ------------------------------------------------------------------------ + // XXX Scope API + // ------------------------------------------------------------------------ + + @Override + public final Instant creationTime() { + return creationTime; + } + + @Override + public final Configuration configuration() { + return configuration; + } + + @Override + public final DSLContext dsl() { + return configuration.dsl(); + } + + @Override + public final Settings settings() { + return configuration.settings(); + } + + @Override + public final SQLDialect dialect() { + return configuration.dialect(); + } + + @Override + public final SQLDialect family() { + return configuration.family(); + } + + @Override + public final Map data() { + if (data == null) + data = new DataMap(); + + return data; + } + + @Override + public final Object data(Object key) { + return data().get(key); + } + + @Override + public final Object data(Object key, Object value) { + return data().put(key, value); + } +} diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractNode.java b/jOOQ/src/main/java/org/jooq/impl/AbstractNode.java index 62632d32c5..86ed96cefe 100644 --- a/jOOQ/src/main/java/org/jooq/impl/AbstractNode.java +++ b/jOOQ/src/main/java/org/jooq/impl/AbstractNode.java @@ -50,7 +50,7 @@ import org.jooq.exception.DataDefinitionException; /** * @author Lukas Eder */ -abstract class AbstractNode> extends AbstractScope implements Node { +abstract class AbstractNode> extends AbstractLazyScope implements Node { final N root; final String id; diff --git a/jOOQ/src/main/java/org/jooq/impl/DefaultCacheContext.java b/jOOQ/src/main/java/org/jooq/impl/DefaultCacheContext.java index f2ee54a2dc..19a6a457ae 100644 --- a/jOOQ/src/main/java/org/jooq/impl/DefaultCacheContext.java +++ b/jOOQ/src/main/java/org/jooq/impl/DefaultCacheContext.java @@ -43,7 +43,7 @@ import org.jooq.Configuration; /** * @author Lukas Eder */ -final class DefaultCacheContext extends AbstractScope implements CacheContext { +final class DefaultCacheContext extends AbstractLazyScope implements CacheContext { private final CacheType cacheType; diff --git a/jOOQ/src/main/java/org/jooq/impl/DefaultConverterContext.java b/jOOQ/src/main/java/org/jooq/impl/DefaultConverterContext.java index a29a14d4ad..8644a1368b 100644 --- a/jOOQ/src/main/java/org/jooq/impl/DefaultConverterContext.java +++ b/jOOQ/src/main/java/org/jooq/impl/DefaultConverterContext.java @@ -45,7 +45,7 @@ import org.jooq.ConverterContext; /** * @author Lukas Eder */ -final class DefaultConverterContext extends AbstractScope implements ConverterContext { +final class DefaultConverterContext extends AbstractLazyScope implements ConverterContext { DefaultConverterContext(Configuration configuration) { super(configuration, new DataMap()); diff --git a/jOOQ/src/main/java/org/jooq/impl/DefaultGeneratorContext.java b/jOOQ/src/main/java/org/jooq/impl/DefaultGeneratorContext.java index c3bf15aaf2..c8bee42496 100644 --- a/jOOQ/src/main/java/org/jooq/impl/DefaultGeneratorContext.java +++ b/jOOQ/src/main/java/org/jooq/impl/DefaultGeneratorContext.java @@ -50,7 +50,7 @@ import org.jooq.Table; /** * @author Lukas Eder */ -final class DefaultGeneratorContext, T> extends AbstractScope implements GeneratorContext { +final class DefaultGeneratorContext, T> extends AbstractLazyScope implements GeneratorContext { final Instant renderTime; final X table;