Re: AW: [eclipselink-dev] setMaxResults(Integer.MAX_VALUE)

[Gordon Yorke <gordon.yorke@xxxxxxxxxx>]

Looks fine to me.  We will have to watch for any users who were depending on the min/max reset upon execution but it is unlikely that anyone would even suspect EclipseLink had this behaviour.
--Gordon

Goerler, Adrian wrote:

Hi Gordon, others,

 

I am proposing a patch for this issue:

 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=301274

 

As it would introduce incompatibilities, I would like to collect some opinions on the proposal.

 

-Adrian

 

Adrian Görler
SAP AG

Pflichtangaben/Mandatory Disclosure Statements: http://www.sap.com/company/legal/impressum.epx

 

Von: eclipselink-dev-bounces@xxxxxxxxxxx [mailto:eclipselink-dev-bounces@xxxxxxxxxxx] Im Auftrag von Gordon Yorke
Gesendet: Montag, 1. Februar 2010 15:20
An: Dev mailing list for Eclipse Persistence Services
Betreff: Re: [eclipselink-dev] setMaxResults(Integer.MAX_VALUE)

 

'-1' is the value used internally to represent unset.  '0' would mean you would get no results.  Seems there is no way to unset max results before executing the query without some inside knowledge.
--Gordon

James Sutherland wrote:

I believe if you set a firstResult, and just don't set the maxResult, you will get unlimited.  I would assume setting maxResult to 0 would have the same affect.

-----Original Message-----
From: Goerler, Adrian [mailto:adrian.goerler@xxxxxxx]
Sent: Monday, February 01, 2010 5:35 AM
To: Gordon Yorke; Schweigkoffer, Rainer
Cc: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] setMaxResults(Integer.MAX_VALUE)

Hi Gordon, Rainer,

 

What’s your opinion on this issue:

 

https://bugs.eclipse.org/bugs/show_bug.cgi?id=301274

 

-Adrian

 

Adrian Görler
SAP AG

Pflichtangaben/Mandatory Disclosure Statements: http://www.sap.com/company/legal/impressum.epx

 

 

 

_______________________________________________

eclipselink-dev mailing list

eclipselink-dev@xxxxxxxxxxx

https://dev.eclipse.org/mailman/listinfo/eclipselink-dev