groovy

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

depends upon

New Feature - A new feature of the product, which has yet to be developed. GROOVY-5405 Named parameters for groovy.sql.Sql

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Frankfurter added a comment -

Created pull request https://github.com/groovy/groovy-core/pull/38

Show
Michael Frankfurter added a comment - Created pull request https://github.com/groovy/groovy-core/pull/38
Hide
Permalink
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
Permalink
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
Permalink
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.

People

  • Assignee:
    Paul King
    Reporter:
    Michael Frankfurter
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: