HHH-16814 add getRootList()

The method getRoots() returning a Set is really inconvenient

hibernate-core/src/main/java/org/hibernate/query/criteria/JpaCriteriaQuery.java

@@ -13,6 +13,7 @@
 import jakarta.persistence.criteria.Order;
 import jakarta.persistence.criteria.ParameterExpression;
 import jakarta.persistence.criteria.Predicate;
+import jakarta.persistence.criteria.Root;
 import jakarta.persistence.criteria.Selection;
 import jakarta.persistence.metamodel.EntityType;
 
@@ -49,6 +50,11 @@ public interface JpaCriteriaQuery<T> extends CriteriaQuery<T>, JpaQueryableCrite
 	// ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 	// Accessors
 
+	/**
+	 * Return the {@linkplain #getRoots() roots} as a list.
+	 */
+	List<Root<?>> getRootList();
+
 	@Override
 	@SuppressWarnings("unchecked")
 	default List<Order> getOrderList() {

hibernate-core/src/main/java/org/hibernate/query/criteria/JpaQueryStructure.java

@@ -46,6 +46,8 @@ public interface JpaQueryStructure<T> extends JpaQueryPart<T> {
 
 	Set<? extends JpaRoot<?>> getRoots();
 
+	List<? extends JpaRoot<?>> getRootList();
+
 	JpaQueryStructure addRoot(JpaRoot<?> root);
 
 

hibernate-core/src/main/java/org/hibernate/query/sqm/tree/select/AbstractSqmSelectQuery.java

@@ -211,6 +211,10 @@ public Set<Root<?>> getRoots() {
 		return (Set) getQuerySpec().getRoots();
 	}
 
+	public List<Root<?>> getRootList() {
+		return (List) getQuerySpec().getRootList();
+	}
+
 	@Override
 	public <X> SqmRoot<X> from(Class<X> entityClass) {
 		return addRoot(

hibernate-core/src/main/java/org/hibernate/query/sqm/tree/select/SqmQuerySpec.java

@@ -318,6 +318,12 @@ public Set<SqmRoot<?>> getRoots() {
 		return new HashSet<>( getFromClause().getRoots() );
 	}
 
+	@Override
+	public List<SqmRoot<?>> getRootList() {
+		assert getFromClause() != null;
+		return getFromClause().getRoots();
+	}
+
 	@Override
 	public SqmQuerySpec<T> addRoot(JpaRoot<?> root) {
 		if ( getFromClause() == null ) {