CAY-2836 ObjectSelect.selectCount() throws if a query contains ordering

Author: stariy95

RELEASE-NOTES.txt

@@ -22,6 +22,7 @@ CAY-2897 Add no-op default implementations to the GraphChangeHandler interface
 Bug Fixes:
 
 CAY-2701 MySQL DST-related LocalDateTime issues
+CAY-2836 ObjectSelect.selectCount() throws if a query contains ordering
 CAY-2871 QualifierTranslator breaks on a relationship with a compound FK
 CAY-2872 CayenneModeler "Documentation" link is broken
 CAY-2876 Memory leak in the ObjectStore

cayenne/src/main/java/org/apache/cayenne/query/ObjectSelect.java

@@ -328,7 +328,10 @@ public ObjectSelect<T> distinct() {
      * @return count of rows
      */
     public long selectCount(ObjectContext context) {
-        return count().selectOne(context);
+        ColumnSelect<Long> count = count();
+        // reset orderings, see CAY-2836
+        count.orderings = List.of();
+        return count.selectOne(context);
     }
 
     @Override

cayenne/src/test/java/org/apache/cayenne/query/ObjectSelect_RunIT.java

@@ -246,4 +246,12 @@ public void test_CAY_2092() {
 			assertNotNull(artist);
 		}
 	}
+
+	@Test
+	public void test_CAY_2836_countWithOrdering() {
+		long count = ObjectSelect.query(Artist.class)
+				.orderBy(Artist.ARTIST_NAME.asc())
+				.selectCount(context);
+		assertEquals(20, count);
+	}
 }