Hi,I don't like the ignore option. You get the wrong answer but we don't tell you? No, thanks.
I like allow, warn, and error (even though error isn't a verb).We also need to carefully read the JPA TCK compatibility rules. "Extensions" that the specification *requires* to throw an error are problematic.
Craig On Jan 16, 2008, at 11:27 AM, Albert Lee wrote:
After some thought, I think it would be more intuitive to use [allow|warn|error], where: if extension is specified, allow = it will be honored, warning = it will be processed but post a warning message and error = an exception will be thrown.ignore can also be a option as a toleration mode, that is application withextension will be toleratedignore = extension is NOT honored but no warning message or exception willbe thrown. Albert Lee. On Jan 16, 2008 11:11 AM, Pinaki Poddar <[EMAIL PROTECTED]> wrote:Liked this one better openjpa.extensions.JPQL = [true|false|warn] On 'ignore' option I was thinking of a choice where the runtime continues working when extended JPQL is used but ignores all the extended clasues. On second thought though that does not look good to me. Instead of 'All' how about? openjpa.extensions.* = [true|false|warn]If some of the extensions become part of JPA spec in future, then stillsome features are more likely to remain exclusive to OpenJPA so the property will reatin its significance. -----Original Message----- From: Albert Lee [mailto:[EMAIL PROTECTED] Sent: Wednesday, January 16, 2008 10:35 AM To: [email protected] Subject: Re: JPQL extensionsIs JPQL the ONLY spec extension we have in mind? Do we just want 'none' or 'all' option or allow more granular enable when more extensions areadded in the future? How about openjpa.extensions.All=[true|false|warn] openjpa.extensions.JPQL=[true|false|warn] openjpa.extensions.Xxxxxx=[true|false|warn] <-- future extensionsWhat about if an extension gets adopted in future JPA spec, then how arethe options get interpreted? Albert Lee. On Jan 16, 2008 1:27 AM, Patrick Linskey <[EMAIL PROTECTED]> wrote:How would 'ignore' and 'false' differ?Also, your naming would presumably restrict the property to just JPQL. What about a broader toggle to control strict JPA compliance? It seemslike if someone wants strict compliance, they're going to want it everywhere, not just in JPQL. -Patrick On Jan 15, 2008 8:05 PM, Pinaki Poddar <[EMAIL PROTECTED]> wrote:openjpa.jpql.Extensions : true|false|ignore|warn -----Original Message----- From: Patrick Linskey [mailto:[EMAIL PROTECTED] Sent: Tuesday, January 15, 2008 10:03 PM To: [email protected] Subject: JPQL extensions Hi, Earlier today, I modified the JPQL parser to allow some extensionsto JPQL. I expect that there is other non-standard extended behavior(JPQL and otherwise) in OpenJPA. It seems like it might be desirableto have a property that controls whether or not OpenJPA allowsextended behavior.I was thinking that a property like so could be useful: openjpa.JPAExtensions: true | false | warn Thoughts? -Patrick -- Patrick Linskey 202 669 5907 Notice: This email message, together with any attachments, may containinformation of BEA Systems, Inc., its subsidiaries and affiliated entities, that may be confidential, proprietary,copyrighted and/or legally privileged, and is intended solely for theuse of the individual or entity named in this message. If you are notthe intended recipient, and have received this message in error, please immediately return this by email and then delete it.-- Patrick Linskey 202 669 5907-- Albert Lee.Notice: This email message, together with any attachments, may contain information of BEA Systems, Inc., its subsidiaries and affiliated entities, that may be confidential, proprietary, copyrighted and/or legally privileged, and is intended solely for the use of the individual or entity named in this message. If you are not the intended recipient, and have received this message in error, please immediately return this by emailand then delete it.-- Albert Lee.
Craig Russell Architect, Sun Java Enterprise System http://java.sun.com/products/jdo 408 276-5638 mailto:[EMAIL PROTECTED] P.S. A good JDO? O, Gasp!
smime.p7s
Description: S/MIME cryptographic signature
