From b2586d5f2a612a54782749466703b2596efb2001 Mon Sep 17 00:00:00 2001
From: Gavin <gavin@hibernate.org>
Date: Thu, 29 Dec 2022 20:25:12 +0100
Subject: [PATCH] clean up very obsolete doc of SessionFactoryImpl

---
 .../internal/SessionFactoryImpl.java          | 19 ++++++++-----------
 .../org/hibernate/internal/SessionImpl.java   |  7 ++++---
 .../internal/StatelessSessionImpl.java        |  7 ++++---
 3 files changed, 16 insertions(+), 17 deletions(-)

diff --git a/hibernate-core/src/main/java/org/hibernate/internal/SessionFactoryImpl.java b/hibernate-core/src/main/java/org/hibernate/internal/SessionFactoryImpl.java
index 5950477d86..145020f2ea 100644
--- a/hibernate-core/src/main/java/org/hibernate/internal/SessionFactoryImpl.java
+++ b/hibernate-core/src/main/java/org/hibernate/internal/SessionFactoryImpl.java
@@ -134,21 +134,18 @@ import static org.hibernate.internal.util.config.ConfigurationHelper.getBoolean;
 import static org.hibernate.query.QueryLogging.QUERY_MESSAGE_LOGGER;
 
 /**
- * Concrete implementation of the {@code SessionFactory} interface. Has the following
- * responsibilities:
+ * Concrete implementation of the {@link SessionFactory} API.
+ * <p>
+ * Exposes two interfaces:
  * <ul>
- * <li>caches configuration settings (immutably)
- * <li>caches "compiled" mappings ie. {@code EntityPersister}s and
- *     {@code CollectionPersister}s (immutable)
- * <li>caches "compiled" queries (memory sensitive cache)
- * <li>manages {@code PreparedStatement}s
- * <li> delegates JDBC {@code Connection} management to the {@code ConnectionProvider}
- * <li>factory for instances of {@code SessionImpl}
+ * <li>{@link SessionFactory} to the application, and</li>
+ * <li>{@link SessionImplementor} (an SPI interface) to other subsystems.</li>
  * </ul>
  * <p>
+ * This class is not thread-safe.
  * This class must appear immutable to clients, even if it does all kinds of caching
- * and pooling under the covers. It is crucial that the class is not only thread
- * safe, but also highly concurrent. Synchronization must be used extremely sparingly.
+ * and pooling under the covers. It is crucial that the class is not only thread-safe,
+ * but also highly concurrent. Synchronization must be used extremely sparingly.
  *
  * @author Gavin King
  * @author Steve Ebersole
diff --git a/hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java b/hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java
index 49d2b85dbd..7994d046ce 100644
--- a/hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java
+++ b/hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java
@@ -165,9 +165,10 @@ import static org.hibernate.proxy.HibernateProxy.extractLazyInitializer;
 /**
  * Concrete implementation of the {@link Session} API.
  * <p>
- * Exposes two interfaces:<ul>
- * <li>{@link Session} to the application</li>
- * <li>{@link SessionImplementor} and {@link EventSource} to other Hibernate components (SPI)</li>
+ * Exposes two interfaces:
+ * <ul>
+ * <li>{@link Session} to the application, and
+ * <li>{@link SessionImplementor} and {@link EventSource} (both SPI interfaces) to other subsystems.
  * </ul>
  * <p>
  * This class is not thread-safe.
diff --git a/hibernate-core/src/main/java/org/hibernate/internal/StatelessSessionImpl.java b/hibernate-core/src/main/java/org/hibernate/internal/StatelessSessionImpl.java
index 1f93ca4630..3501c9b0f0 100644
--- a/hibernate-core/src/main/java/org/hibernate/internal/StatelessSessionImpl.java
+++ b/hibernate-core/src/main/java/org/hibernate/internal/StatelessSessionImpl.java
@@ -47,9 +47,10 @@ import static org.hibernate.proxy.HibernateProxy.extractLazyInitializer;
 /**
  * Concrete implementation of the {@link StatelessSession} API.
  * <p>
- * Exposes two interfaces:<ul>
- * <li>{@link StatelessSession} to the application</li>
- * <li>{@link org.hibernate.engine.spi.SharedSessionContractImplementor} to other Hibernate components (SPI)</li>
+ * Exposes two interfaces:
+ * <ul>
+ * <li>{@link StatelessSession} to the application, and
+ * <li>{@link org.hibernate.engine.spi.SharedSessionContractImplementor} (an SPI interface) to other subsystems.
  * </ul>
  * <p>
  * This class is not thread-safe.