FAQ

On Jan 21, 2010, at 11:58 AM, andrey@apache.org wrote:

Modified: cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/
src/main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt
URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt?rev=901627&r1=901626&r2=901627&view=diff
=
=
=
=
=
=
=
=
======================================================================
--- cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/
jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt (original)
+++ cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/
jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt Thu Jan 21 09:58:38
2010
@@ -1208,7 +1208,7 @@

void pattern_value() #PatternValue : { }
{
- input_parameter() | string_literal()
+ string_expression()
[(<ESCAPE> escape_character())]
}

-1.

This significantly expands the definition of "pattern value" beyond
EJBQL spec. The spec on page 93 says:

"The pattern_value is a string literal or a string-valued input
parameter".

I don't see a pressing need for us to deviate from the spec here. A
simple workaround that Andreas seems to have been using already is to
uppercase the pattern Java String. So it was correct before, and now
it allows things like subqueries to be a "pattern_value". SO IMO
CAY-1369 was not a bug.

Andrus

Search Discussions

  • Andrus Adamchik at Jan 21, 2010 at 4:08 pm
    So more specifically, I think if we want to provide likeIgnoreCase
    functionality in CQL (extends EJBQL), we'd rather add a new operator.

    Andrus
    On Jan 21, 2010, at 5:57 PM, Andrus Adamchik wrote:

    On Jan 21, 2010, at 11:58 AM, andrey@apache.org wrote:

    Modified: cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/
    src/main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt
    URL: http://svn.apache.org/viewvc/cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt?rev=901627&r1=901626&r2=901627&view=diff
    =
    =
    =
    =
    =
    =
    =
    =
    =
    =====================================================================
    --- cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/
    main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt (original)
    +++ cayenne/main/trunk/framework/cayenne-jdk1.5-unpublished/src/
    main/jjtree/org/apache/cayenne/ejbql/EJBQLParser.jjt Thu Jan 21
    09:58:38 2010
    @@ -1208,7 +1208,7 @@

    void pattern_value() #PatternValue : { }
    {
    - input_parameter() | string_literal()
    + string_expression()
    [(<ESCAPE> escape_character())]
    }

    -1.

    This significantly expands the definition of "pattern value" beyond
    EJBQL spec. The spec on page 93 says:

    "The pattern_value is a string literal or a string-valued input
    parameter".

    I don't see a pressing need for us to deviate from the spec here. A
    simple workaround that Andreas seems to have been using already is
    to uppercase the pattern Java String. So it was correct before, and
    now it allows things like subqueries to be a "pattern_value". SO IMO
    CAY-1369 was not a bug.

    Andrus

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupdev @
categoriescayenne
postedJan 21, '10 at 3:58p
activeJan 21, '10 at 4:08p
posts2
users1
websitecayenne.apache.org

1 user in discussion

Andrus Adamchik: 2 posts

People

Translate

site design / logo © 2022 Grokbase