[ 
http://issues.apache.org/jira/browse/JDO-370?page=comments#action_12375360 ] 

Craig Russell commented on JDO-370:
-----------------------------------

The patch looks good.

I'd have to look more carefully, but generally by the specification, it's not 
illegal to declare parameters that are not used in the query filter. So the 
mismatch would not be discovered since there is no need to look at the 
parameter values. But this case isn't covered by the spec, so it is good to fix 
the test case.


> Valid query seems invalid in SetterReplacePreviousValues
> --------------------------------------------------------
>
>          Key: JDO-370
>          URL: http://issues.apache.org/jira/browse/JDO-370
>      Project: JDO
>         Type: Bug

>   Components: tck20
>     Reporter: Ilan Kirsh
>     Assignee: Michael Bouschen
>  Attachments: JDO-370.patch
>
> Probably I am missing something, but as far as I can see the second valid 
> query in org.apache.jdo.tck.query.api.SetterReplacePreviousValues, has a 
> String parameter and receives a long argument, so it should throw an 
> exception, isn't it?

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira

Reply via email to