Groovy
  1. Groovy
  2. GROOVY-5401

Extend groovy.sql.DataSet with paging capabilities

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta-2, 1.8.6
    • Fix Version/s: 2.0-beta-3, 1.8.7
    • Component/s: SQL processing
    • Labels:
      None

      Description

      A question at stackoverflow.com/questions/9949884/ stated out paging is not supported in DataSet class which is correct. Digging into the code made this a valid statement but also revealed, it would be easy to add. I just created this task to have an ID to refer to in my future pull request.

        Issue Links

          Activity

          Michael Frankfurter created issue -
          Hide
          Michael Frankfurter added a comment -
          Show
          Michael Frankfurter added a comment - Created pull request https://github.com/groovy/groovy-core/pull/38
          Paul King made changes -
          Field Original Value New Value
          Link This issue depends upon GROOVY-5405 [ GROOVY-5405 ]
          Hide
          Paul King added a comment -

          Just an update: I am waiting until we finalise plans for GROOVY-5405 to make sure it doesn't impact this issue - which it shouldn't, then I think the pull request can be applied.

          Show
          Paul King added a comment - Just an update: I am waiting until we finalise plans for GROOVY-5405 to make sure it doesn't impact this issue - which it shouldn't, then I think the pull request can be applied.
          Hide
          Paul King added a comment -

          Added. Thanks for the suggestion and pull request. If you get a chance to update the stackoverflow discussion, that would be great. Thanks.

          Show
          Paul King added a comment - Added. Thanks for the suggestion and pull request. If you get a chance to update the stackoverflow discussion, that would be great. Thanks.
          Paul King made changes -
          Assignee Paul King [ paulk ]
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Michael Frankfurter added a comment -

          Thanks for pulling it in. I will update the stackoverflow discussion after 1.8.7 is released. Regards, Michael.

          Show
          Michael Frankfurter added a comment - Thanks for pulling it in. I will update the stackoverflow discussion after 1.8.7 is released. Regards, Michael.
          Paul King made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
          Mark Thomas made changes -
          Workflow jira [ 12734324 ] Default workflow, editable Closed status [ 12746083 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
          Mark Thomas made changes -
          Workflow jira [ 12971798 ] Default workflow, editable Closed status [ 12979605 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Resolved Resolved
          6d 21h 16m 1 Paul King 16/Apr/12 04:31
          Resolved Resolved Closed Closed
          26d 23h 58m 1 Paul King 13/May/12 04:30

            People

            • Assignee:
              Paul King
              Reporter:
              Michael Frankfurter
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development