groovy
  1. groovy
  2. GROOVY-5401

Extend groovy.sql.DataSet with paging capabilities

    Details

    • Type: Improvement Improvement
    • Status: Closed 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
    • Number of attachments :
      0

      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

          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 -
          Status Open [ 1 ] Resolved [ 5 ]
          Assignee Paul King [ paulk ]
          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 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: