Uploaded image for project: 'Couchbase Java Client'
  1. Couchbase Java Client
  2. JCBC-49

Query limit does not work with paged queries

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1dp
    • Fix Version/s: 1.1-dp3
    • Component/s: Core
    • Security Level: Public
    • Labels:
      None
    • Environment:
      all java

      Description

      Query limit property does not work with paged queries. Paginator overwrites the limit with the page size without saving the original and a counter is not kept.

      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        Hide
        mikew Mike Wiederhold added a comment -

        From this issue it sounds like you want to add the following:

        1. The limit in the Query class should override the numDocs parameter in the Paginator class.
        2. A counter of the number of documents we have iterated through the paginator should be kept.

        Am I correct here?

        Show
        mikew Mike Wiederhold added a comment - From this issue it sounds like you want to add the following: 1. The limit in the Query class should override the numDocs parameter in the Paginator class. 2. A counter of the number of documents we have iterated through the paginator should be kept. Am I correct here?
        Hide
        mikew Mike Wiederhold added a comment -

        I think I'm going to throw an exception in the case that a user defines their own limit parameter.

        Show
        mikew Mike Wiederhold added a comment - I think I'm going to throw an exception in the case that a user defines their own limit parameter.
        Hide
        mikew Mike Wiederhold added a comment -

        Query limit is now honored in paginated queries.

        Show
        mikew Mike Wiederhold added a comment - Query limit is now honored in paginated queries.

          People

          • Assignee:
            mikew Mike Wiederhold
            Reporter:
            SteveC Steven Cooke
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes