[ 
https://issues.apache.org/jira/browse/OPENJPA-990?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Donald Woods updated OPENJPA-990:
---------------------------------

    Attachment: OPENJPA-990.patch

Patch that will cause setHint() for javax.persistence.lock/query.timeout to 
return a OpenJPA ArgumentException (which extends the spec required 
IllegalArgumentException) if the provided timeout is invalid (timeout < -1), as 
-1 is allowed by OpenJPA to denote no timeout (the same as providing 0).
Three junit tests were added to test for the new IllegalArgumentException 
behavior.


> setHint should return IllegalArgumentException for invalid query/lock timeout 
> values
> ------------------------------------------------------------------------------------
>
>                 Key: OPENJPA-990
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-990
>             Project: OpenJPA
>          Issue Type: Sub-task
>          Components: query
>    Affects Versions: 2.0.0
>            Reporter: Donald Woods
>            Assignee: Donald Woods
>             Fix For: 2.0.0
>
>         Attachments: OPENJPA-990.patch
>
>
> JPA2 spec notes that Query.setHint() should throw a IllegalArgumentException 
> if a supplied hint value is not valid for a given implementation.
> For lock/query timeout, any user supplied value < -1 is invalid and should 
> throw an exception, which is not occurring.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to