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

          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
          Mark Thomas made changes -
          Workflow jira [ 12971798 ] Default workflow, editable Closed status [ 12979605 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 02:11:23 UTC 2015 [ 1428286283443 ]
          Mark Thomas made changes -
          Workflow jira [ 12734324 ] Default workflow, editable Closed status [ 12746083 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 13:32:57 UTC 2015 [ 1428240777691 ]
          Paul King made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          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 -
          Assignee Paul King [ paulk ]
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          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.
          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.
          Paul King made changes -
          Field Original Value New Value
          Link This issue depends upon GROOVY-5405 [ GROOVY-5405 ]
          Hide
          Michael Frankfurter added a comment -
          Show
          Michael Frankfurter added a comment - Created pull request https://github.com/groovy/groovy-core/pull/38
          Michael Frankfurter created issue -

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development