Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0-M2
    • Fix Version/s: 2.0.0-M2
    • Component/s: query
    • Labels:
      None

      Description

      Include additional junit tests for setting the query timeout property via Persistence.createEntityManagerFactory(Map properties) and PU properties in persistence.xml.
      Need to add a test showing that a PU or Map provided query timeout does not affect EM.find()/findAll() operations.

      1. OPENJPA-963.patch
        42 kB
        Donald Woods

        Issue Links

          Activity

          Hide
          Donald Woods added a comment -

          Depends on persistence3.xml added by OPENJPA-972

          Show
          Donald Woods added a comment - Depends on persistence3.xml added by OPENJPA-972
          Hide
          Donald Woods added a comment -

          Patch that adds addition PU/Map tests for query timeout and cleans up some existing tests and comments provided by OPENJPA-878 and OPENJPA-972.
          Updates DBDictionary.setQueryTimeout() by:
          1) handling special timeout=-1 case, by converting it to JDBC expected 0
          2) adding new log message for timeout < -1, which will not be set
          3) adding new setQueryTimeoutDB() method as requested by Fay

          Show
          Donald Woods added a comment - Patch that adds addition PU/Map tests for query timeout and cleans up some existing tests and comments provided by OPENJPA-878 and OPENJPA-972 . Updates DBDictionary.setQueryTimeout() by: 1) handling special timeout=-1 case, by converting it to JDBC expected 0 2) adding new log message for timeout < -1, which will not be set 3) adding new setQueryTimeoutDB() method as requested by Fay
          Hide
          Donald Woods added a comment -

          Fay, can you review and commit this patch, as it includes your requested DBDictionary.setQueryTimeoutDB() method?

          Show
          Donald Woods added a comment - Fay, can you review and commit this patch, as it includes your requested DBDictionary.setQueryTimeoutDB() method?
          Hide
          Donald Woods added a comment -

          Committed to trunk as Rev756193 by Fay.

          Show
          Donald Woods added a comment - Committed to trunk as Rev756193 by Fay.

            People

            • Assignee:
              Donald Woods
              Reporter:
              Donald Woods
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development