move classes to correct package (instead of "iternal")

and fix lots of warnings

Author: gavinking

hibernate-reactive-core/src/main/java/org/hibernate/reactive/mutiny/impl/MutinySessionImpl.java

@@ -33,7 +33,7 @@
 import org.hibernate.reactive.mutiny.Mutiny.Query;
 import org.hibernate.reactive.mutiny.Mutiny.SelectionQuery;
 import org.hibernate.reactive.pool.ReactiveConnection;
-import org.hibernate.reactive.query.sqm.iternal.ReactiveQuerySqmImpl;
+import org.hibernate.reactive.query.sqm.internal.ReactiveQuerySqmImpl;
 import org.hibernate.reactive.session.ReactiveConnectionSupplier;
 import org.hibernate.reactive.session.ReactiveQueryProducer;
 import org.hibernate.reactive.session.ReactiveSession;

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/spi/ReactiveAbstractSelectionQuery.java

@@ -30,8 +30,8 @@
 import org.hibernate.query.sqm.tree.select.SqmSelectStatement;
 import org.hibernate.reactive.logging.impl.Log;
 import org.hibernate.reactive.logging.impl.LoggerFactory;
-import org.hibernate.reactive.query.sqm.iternal.AggregatedSelectReactiveQueryPlan;
-import org.hibernate.reactive.query.sqm.iternal.ConcreteSqmSelectReactiveQueryPlan;
+import org.hibernate.reactive.query.sqm.internal.AggregatedSelectReactiveQueryPlan;
+import org.hibernate.reactive.query.sqm.internal.ConcreteSqmSelectReactiveQueryPlan;
 import org.hibernate.reactive.query.sqm.spi.ReactiveSelectQueryPlan;
 import org.hibernate.sql.results.internal.TupleMetadata;
 
@@ -52,9 +52,9 @@ public class ReactiveAbstractSelectionQuery<R> {
 
 	private final Supplier<QueryOptions> queryOptionsSupplier;
 
-	private SharedSessionContractImplementor session;
+	private final SharedSessionContractImplementor session;
 	private final Supplier<CompletionStage<List<R>>> doList;
-	private final Supplier<SqmStatement> getStatement;
+	private final Supplier<SqmStatement<?>> getStatement;
 
 	private final Supplier<TupleMetadata> getTupleMetadata;
 
@@ -66,7 +66,7 @@ public class ReactiveAbstractSelectionQuery<R> {
 	private final Runnable beforeQuery;
 
 	private final Consumer<Boolean> afterQuery;
-	private Function<List<R>, R> uniqueElement;
+	private final Function<List<R>, R> uniqueElement;
 	private final InterpretationsKeySource interpretationsKeySource;
 
 	// I'm sure we can avoid some of this by making some methods public in ORM,
@@ -75,7 +75,7 @@ public ReactiveAbstractSelectionQuery(
 			InterpretationsKeySource interpretationKeySource,
 			SharedSessionContractImplementor session,
 			Supplier<CompletionStage<List<R>>> doList,
-			Supplier<SqmStatement> getStatement,
+			Supplier<SqmStatement<?>> getStatement,
 			Supplier<TupleMetadata> getTupleMetadata,
 			Supplier<DomainParameterXref> getDomainParameterXref,
 			Supplier<Class<R>> getResultType,
@@ -98,40 +98,12 @@ public ReactiveAbstractSelectionQuery(
 				interpretationKeySource
 		);
 	}
-//
-//	public ReactiveAbstractSelectionQuery(
-//			DomainQueryExecutionContext domainQueryExecutionContext,
-//			SharedSessionContractImplementor session,
-//			Supplier<CompletionStage<List<R>>> doList,
-//			Supplier<SqmStatement> getStatement,
-//			Supplier<TupleMetadata> getTupleMetadata,
-//			Supplier<DomainParameterXref> getDomainParameterXref,
-//			Supplier<Class<R>> getResultType,
-//			Supplier<String> getQueryString,
-//			Runnable beforeQuery,
-//			Consumer<Boolean> afterQuery,
-//			Function<List<R>, R> uniqueElement) {
-//		this(
-//				domainQueryExecutionContext::getQueryOptions,
-//				session,
-//				doList,
-//				getStatement,
-//				getTupleMetadata,
-//				getDomainParameterXref,
-//				getResultType,
-//				getQueryString,
-//				beforeQuery,
-//				afterQuery,
-//				uniqueElement,
-//				null
-//		);
-//	}
 
 	public ReactiveAbstractSelectionQuery(
 			Supplier<QueryOptions> queryOptionsSupplier,
 			SharedSessionContractImplementor session,
 			Supplier<CompletionStage<List<R>>> doList,
-			Supplier<SqmStatement> getStatement,
+			Supplier<SqmStatement<?>> getStatement,
 			Supplier<TupleMetadata> getTupleMetadata,
 			Supplier<DomainParameterXref> getDomainParameterXref,
 			Supplier<Class<R>> getResultType,
@@ -292,7 +264,7 @@ private CompletionStage<List<R>> doReactiveList() {
 		return doList.get();
 	}
 
-	public SqmStatement getSqmStatement() {
+	public SqmStatement<?> getSqmStatement() {
 		return getStatement.get();
 	}
 
@@ -329,7 +301,7 @@ public List<R> list() {
 	}
 
 	public Stream<R> getResultStream() {
-		throw LOG.nonReactiveMethodCall( "<no alterative>" );
+		throw LOG.nonReactiveMethodCall( "<no alternative>" );
 	}
 
 	public R uniqueResult() {

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/spi/ReactiveNativeSelectQueryPlan.java

@@ -16,7 +16,7 @@
 import org.hibernate.reactive.logging.impl.LoggerFactory;
 import org.hibernate.reactive.query.sqm.spi.ReactiveSelectQueryPlan;
 
-public interface ReactiveNativeSelectQueryPlan<T> extends NativeSelectQueryPlan, ReactiveSelectQueryPlan {
+public interface ReactiveNativeSelectQueryPlan<T> extends NativeSelectQueryPlan<T>, ReactiveSelectQueryPlan<T> {
 	Log LOG = LoggerFactory.make( Log.class, MethodHandles.lookup() );
 
 	/**

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sql/internal/ReactiveNativeQueryImpl.java

@@ -62,11 +62,6 @@ public class ReactiveNativeQueryImpl<R> extends NativeQueryImpl<R>
 
 	private final ReactiveAbstractSelectionQuery<R> selectionQueryDelegate;
 
-	public ReactiveNativeQueryImpl(String memento, SharedSessionContractImplementor session, String[] querySpaces) {
-		this( memento, session );
-		addQuerySpaces( querySpaces );
-	}
-
 	public ReactiveNativeQueryImpl(String memento, SharedSessionContractImplementor session) {
 		super( memento, session );
 		this.selectionQueryDelegate = createSelectionQueryDelegate( session );

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sql/internal/ReactiveNativeSelectQueryPlanImpl.java

@@ -36,10 +36,10 @@
 import static java.util.Collections.emptyList;
 import static org.hibernate.reactive.util.impl.CompletionStages.completedFuture;
 
-public class ReactiveNativeSelectQueryPlanImpl<R> extends NativeSelectQueryPlanImpl implements ReactiveNativeSelectQueryPlan {
+public class ReactiveNativeSelectQueryPlanImpl<R> extends NativeSelectQueryPlanImpl<R> implements ReactiveNativeSelectQueryPlan<R> {
 
 	private final Set<String> affectedTableNames;
-	private String sql;
+	private final String sql;
 	private final List<ParameterOccurrence> parameterList;
 	private final JdbcValuesMappingProducer resultSetMapping;
 

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sql/spi/ReactiveNamedSqmQueryMemento.java

@@ -20,8 +20,8 @@
 import org.hibernate.query.sqm.spi.NamedSqmQueryMemento;
 import org.hibernate.query.sqm.tree.SqmStatement;
 import org.hibernate.query.sqm.tree.select.SqmSelectStatement;
-import org.hibernate.reactive.query.sqm.iternal.ReactiveQuerySqmImpl;
-import org.hibernate.reactive.query.sqm.iternal.ReactiveSqmSelectionQueryImpl;
+import org.hibernate.reactive.query.sqm.internal.ReactiveQuerySqmImpl;
+import org.hibernate.reactive.query.sqm.internal.ReactiveSqmSelectionQueryImpl;
 
 /**
  * @see org.hibernate.query.sql.spi.NamedNativeQueryMemento

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/iternal/AggregatedSelectReactiveQueryPlan.java renamed to hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/internal/AggregatedSelectReactiveQueryPlan.java

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright: Red Hat Inc. and Hibernate Authors
  */
-package org.hibernate.reactive.query.sqm.iternal;
+package org.hibernate.reactive.query.sqm.internal;
 
 import java.util.ArrayList;
 import java.util.List;

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/iternal/ConcreteSqmSelectReactiveQueryPlan.java renamed to hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/internal/ConcreteSqmSelectReactiveQueryPlan.java

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright: Red Hat Inc. and Hibernate Authors
  */
-package org.hibernate.reactive.query.sqm.iternal;
+package org.hibernate.reactive.query.sqm.internal;
 
 import java.util.List;
 import java.util.Map;
@@ -119,10 +119,10 @@ public ScrollableResultsImplementor<R> performScroll(ScrollMode scrollMode, Doma
 	public CompletionStage<List<R>> reactivePerformList(DomainQueryExecutionContext executionContext) {
 		return executionContext.getQueryOptions().getEffectiveLimit().getMaxRowsJpa() == 0
 				? completedFuture( emptyList() )
-				: withCacheableSqmInterpretation( executionContext, null, listInterpreter );
+				: withCacheableSqmInterpretation( executionContext, listInterpreter );
 	}
 
-	private <T, X> CompletionStage<T> withCacheableSqmInterpretation(DomainQueryExecutionContext executionContext, X context, SqmInterpreter<T, X> interpreter) {
+	private <T, X> CompletionStage<T> withCacheableSqmInterpretation(DomainQueryExecutionContext executionContext, SqmInterpreter<T, X> interpreter) {
 		// NOTE : VERY IMPORTANT - intentional double-lock checking
 		//		The other option would be to leverage `java.util.concurrent.locks.ReadWriteLock`
 		//		to protect access.  However, synchronized is much simpler here.  We will verify
@@ -162,7 +162,7 @@ private <T, X> CompletionStage<T> withCacheableSqmInterpretation(DomainQueryExec
 			jdbcParameterBindings = createJdbcParameterBindings( localCopy, executionContext );
 		}
 
-		return interpreter.interpret( context, executionContext, localCopy, jdbcParameterBindings );
+		return interpreter.interpret( null, executionContext, localCopy, jdbcParameterBindings );
 	}
 
 	private JdbcParameterBindings createJdbcParameterBindings(CacheableSqmInterpretation sqmInterpretation, DomainQueryExecutionContext executionContext) {
@@ -296,13 +296,5 @@ Map<QueryParameterImplementor<?>, Map<SqmParameter<?>, List<JdbcParametersList>>
 		public Map<SqmParameter<?>, MappingModelExpressible<?>> getSqmParameterMappingModelTypes() {
 			return sqmParameterMappingModelTypes;
 		}
-
-		JdbcParameterBindings getFirstParameterBindings() {
-			return firstParameterBindings;
-		}
-
-		void setFirstParameterBindings(JdbcParameterBindings firstParameterBindings) {
-			this.firstParameterBindings = firstParameterBindings;
-		}
 	}
 }

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/iternal/ReactiveMultiTableDeleteQueryPlan.java renamed to hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/internal/ReactiveMultiTableDeleteQueryPlan.java

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright: Red Hat Inc. and Hibernate Authors
  */
-package org.hibernate.reactive.query.sqm.iternal;
+package org.hibernate.reactive.query.sqm.internal;
 
 import java.util.concurrent.CompletionStage;
 
@@ -18,12 +18,12 @@
  * @see org.hibernate.query.sqm.internal.MultiTableDeleteQueryPlan
  */
 public class ReactiveMultiTableDeleteQueryPlan implements ReactiveNonSelectQueryPlan {
-	private final SqmDeleteStatement sqmDelete;
+	private final SqmDeleteStatement<?> sqmDelete;
 	private final DomainParameterXref domainParameterXref;
 	private final ReactiveSqmMultiTableMutationStrategy deleteStrategy;
 
 	public ReactiveMultiTableDeleteQueryPlan(
-			SqmDeleteStatement sqmDelete,
+			SqmDeleteStatement<?> sqmDelete,
 			DomainParameterXref domainParameterXref,
 			ReactiveSqmMultiTableMutationStrategy deleteStrategy) {
 		this.sqmDelete = sqmDelete;

hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/iternal/ReactiveMultiTableInsertQueryPlan.java renamed to hibernate-reactive-core/src/main/java/org/hibernate/reactive/query/sqm/internal/ReactiveMultiTableInsertQueryPlan.java

@@ -3,7 +3,7 @@
  * SPDX-License-Identifier: Apache-2.0
  * Copyright: Red Hat Inc. and Hibernate Authors
  */
-package org.hibernate.reactive.query.sqm.iternal;
+package org.hibernate.reactive.query.sqm.internal;
 
 import java.util.concurrent.CompletionStage;