OpenJPA
  1. OpenJPA
  2. OPENJPA-916

DistributedTemplate is incorrectly setting some attributes on the statements

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.1.0, 1.2.0, 1.3.0, 2.0.0-M2
    • Fix Version/s: 2.0.0-M2
    • Component/s: slice
    • Labels:
      None

      Description

      DistributedTemplate.java in openjpa-slice is not setting the setQueryTimeout() and setMaxRows() properties on its List of statements, but instead is setting the value on setMaxFieldSize() instead.

      1. OPENJPA-916-branches11x.patch
        0.6 kB
        Donald Woods
      2. OPENJPA-916-branches12x.patch
        0.6 kB
        Donald Woods
      3. OPENJPA-916-branches13x.patch
        0.6 kB
        Donald Woods
      4. OPENJPA-916-trunk.patch
        0.6 kB
        Donald Woods

        Activity

        Donald Woods created issue -
        Hide
        Donald Woods added a comment -

        Patch to properly call the setQueryTimeout() and setMaxRows() on the List of Statements.

        Show
        Donald Woods added a comment - Patch to properly call the setQueryTimeout() and setMaxRows() on the List of Statements.
        Donald Woods made changes -
        Field Original Value New Value
        Attachment OPENJPA-916-trunk.patch [ 12399847 ]
        Attachment OPENJPA-916-branches12x.patch [ 12399849 ]
        Attachment OPENJPA-916-branches13x.patch [ 12399848 ]
        Donald Woods made changes -
        Attachment OPENJPA-916-branches11x.patch [ 12399850 ]
        Hide
        Donald Woods added a comment -

        Unassigning so a committer can review and hopefully commit.

        Show
        Donald Woods added a comment - Unassigning so a committer can review and hopefully commit.
        Donald Woods made changes -
        Assignee Donald Woods [ drwoods ]
        Hide
        Pinaki Poddar added a comment -

        Thanks Dan for finding this error.
        I will apply the patch to trunk.

        Show
        Pinaki Poddar added a comment - Thanks Dan for finding this error. I will apply the patch to trunk.
        Pinaki Poddar made changes -
        Assignee Pinaki Poddar [ ppoddar@apache.org ]
        Pinaki Poddar made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Pinaki Poddar made changes -
        Fix Version/s 2.0.0-M1 [ 12313624 ]
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.1.1 [ 12313177 ]
        Fix Version/s 1.3.0 [ 12313326 ]
        Fix Version/s 1.2.1 [ 12313328 ]
        Resolution Fixed [ 1 ]
        Hide
        Donald Woods added a comment -

        Mike, please consider applying to the 1.2.x branch.

        Show
        Donald Woods added a comment - Mike, please consider applying to the 1.2.x branch.
        Donald Woods made changes -
        Assignee Pinaki Poddar [ ppoddar@apache.org ] Michael Dick [ mikedd ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Resolution Fixed [ 1 ]
        Hide
        Michael Dick added a comment -

        Some of the users of the 1.2.x and 1.0.x are conservative about changes, especially without a testcase. For now we'll resolve this issue in the released branches only.

        Show
        Michael Dick added a comment - Some of the users of the 1.2.x and 1.0.x are conservative about changes, especially without a testcase. For now we'll resolve this issue in the released branches only.
        Michael Dick made changes -
        Resolution Fixed [ 1 ]
        Fix Version/s 2.0.0-M1 [ 12313624 ]
        Status Reopened [ 4 ] Resolved [ 5 ]
        Donald Woods made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development