HHH-19348 Change scope of private methods in SessionImpl

Change scopes of some private methods to avoid duplication in Hibernate Reactive.

Related to hibernate/hibernate-reactive#2181

Author: DavideD

hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java

@@ -1102,7 +1102,8 @@ public Object loadFromSecondLevelCache(
 	 * Helper to avoid creating many new instances of {@link PostLoadEvent}.
 	 * It's an allocation hot spot.
 	 */
-	private PostLoadEvent makePostLoadEvent(EntityPersister persister, Object id, Object entity) {
+	// Hibernate Reactive may need to use this
+	protected PostLoadEvent makePostLoadEvent(EntityPersister persister, Object id, Object entity) {
 		final PostLoadEvent event = postLoadEvent;
 		if ( event == null ) {
 			return new PostLoadEvent( id, persister, entity, this );
@@ -1120,7 +1121,8 @@ private PostLoadEvent makePostLoadEvent(EntityPersister persister, Object id, Ob
 	 * Helper to avoid creating many new instances of {@link LoadEvent}.
 	 * It's an allocation hot spot.
 	 */
-	private LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly, LockOptions lockOptions) {
+	// Hibernate Reactive may need to use this
+	protected LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly, LockOptions lockOptions) {
 		final LoadEvent event = loadEvent;
 		if ( event == null ) {
 			return new LoadEvent( id, entityName, lockOptions, this, readOnly );
@@ -1141,7 +1143,8 @@ private LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly,
 	 * Helper to avoid creating many new instances of {@link LoadEvent}.
 	 * It's an allocation hot spot.
 	 */
-	private LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly, boolean isAssociationFetch) {
+	// Hibernate Reactive may need to use this
+	protected LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly, boolean isAssociationFetch) {
 		final LoadEvent event = loadEvent;
 		if ( event == null ) {
 			return new LoadEvent( id, entityName, isAssociationFetch, this, readOnly );
@@ -1158,7 +1161,8 @@ private LoadEvent makeLoadEvent(String entityName, Object id, Boolean readOnly,
 		}
 	}
 
-	private void releasePostLoadEvent(PostLoadEvent event) {
+	// Hibernate Reactive may need to use this
+	protected void releasePostLoadEvent(PostLoadEvent event) {
 		if ( postLoadEvent == null ) {
 			event.setEntity( null );
 			event.setId( null );
@@ -1167,7 +1171,8 @@ private void releasePostLoadEvent(PostLoadEvent event) {
 		}
 	}
 
-	private void releaseLoadEvent(LoadEvent event) {
+	// Hibernate Reactive may need to use this
+	protected void releaseLoadEvent(LoadEvent event) {
 		if ( loadEvent == null ) {
 			event.setEntityClassName( null );
 			event.setEntityId( null );
@@ -2559,7 +2564,8 @@ public <T> T find(EntityGraph<T> entityGraph, Object primaryKey, FindOption... o
 		return loadAccess.withLoadGraph( graph ).load( primaryKey );
 	}
 
-	private void checkTransactionNeededForLock(LockMode lockMode) {
+	// Hibernate Reactive may need to use this
+	protected void checkTransactionNeededForLock(LockMode lockMode) {
 		// OPTIMISTIC and OPTIMISTIC_FORCE_INCREMENT require a transaction
 		// because they involve a version check at the end of the transaction
 		// All flavors of PESSIMISTIC lock also clearly require a transaction
@@ -2568,7 +2574,8 @@ private void checkTransactionNeededForLock(LockMode lockMode) {
 		}
 	}
 
-	private static Boolean readOnlyHint(Map<String, Object> properties) {
+	// Hibernate Reactive may need to use this
+	protected static Boolean readOnlyHint(Map<String, Object> properties) {
 		if ( properties == null ) {
 			return null;
 		}
@@ -3056,7 +3063,8 @@ private void readObject(ObjectInputStream ois) throws IOException, ClassNotFound
 		eventListenerGroups = getFactory().getEventListenerGroups();
 	}
 
-	private Boolean getReadOnlyFromLoadQueryInfluencers() {
+	// Used by Hibernate reactive
+	protected Boolean getReadOnlyFromLoadQueryInfluencers() {
 		return loadQueryInfluencers != null ? loadQueryInfluencers.getReadOnly() : null;
 	}
 }