[#OPENJPA-165] QueryImpl.setFirstResult does not take already set maxResults into account

OpenJPA

QueryImpl.setFirstResult does not take already set maxResults into account

Created: 09/Mar/07 06:06 PM Updated: 07/Aug/07 11:47 PM

Return to search

Component/s:

query

Affects Version/s:

0.9.7

Fix Version/s:

1.0.0

Time Tracking:

Not Specified

Resolution Date:

07/Aug/07 11:47 PM

Description

« Hide

If you want rows 10-19 of some query, the sequence

q.setFirstResult(10).setMaxResults(10);

behaves as expected. On the other hand

q.setMaxResults(10).setFirstResult(10);

returns the empty list. QueryImpl.setFirstResult should reset the range endpoint. maxResults is a count, not an index.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Kevin Sutter made changes - 13/Mar/07 12:23 PM

Field	Original Value	New Value
Assignee		Kevin Sutter [ kwsutter ]

[ Permlink | « Hide ]

Kevin Sutter added a comment - 13/Mar/07 12:24 PM

I've discussed this problem with Jonathan and we should have a patch available soon.

Kevin Sutter made changes - 13/Mar/07 12:24 PM

Fix Version/s

0.9.7 [ 12312340 ]

Michael Dick made changes - 02/Apr/07 07:54 PM

Fix Version/s		0.9.8 [ 12312446 ]
Fix Version/s	0.9.7 [ 12312340 ]

Patrick Linskey made changes - 07/Aug/07 05:18 PM

Fix Version/s	0.9.8 [ 12312446 ]
Fix Version/s		1.0.0 [ 12312341 ]

Patrick Linskey made changes - 07/Aug/07 11:47 PM

Resolution		Fixed [ 1 ]
Status	Open [ 1 ]	Resolved [ 5 ]

Repository	Revision	Date	User	Message
ASF	#563702	Tue Aug 07 23:47:19 UTC 2007	pcl	~~OPENJPA-165~~
				Files Changed
				MODIFY /openjpa/trunk/openjpa-persistence/src/main/java/org/apache/openjpa/persistence/QueryImpl.java ADD /openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestQueryPagination.java (from /openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/query/TestNewlinesInQueries.java)