Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-6979

skip and limit parameters are not shown on UI for spatial views

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Duplicate
    • Affects Version/s: 2.0
    • Fix Version/s: 3.0
    • Component/s: UI
    • Security Level: Public
    • Labels:
    • Environment:

      Description

      limit and skip parameters should be added automatically to query url on UI for spatial views.
      Right now user cannot see these 2 parameters in query url neither in query parameters pop up (see screenshot)

        Issue Links

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

          Activity

          Hide
          farshid Farshid Ghods (Inactive) added a comment -

          Volker,

          does spatial views support those options ?

          please assign the ticket back to alk after commenting on the ticket

          Show
          farshid Farshid Ghods (Inactive) added a comment - Volker, does spatial views support those options ? please assign the ticket back to alk after commenting on the ticket
          Hide
          vmx Volker Mische added a comment - - edited

          Yes, skip and limit is supported now.

          Please also note that the bbox parameter is now optional (and not required as before). If it isn't gived the full result set it returned.

          Show
          vmx Volker Mische added a comment - - edited Yes, skip and limit is supported now. Please also note that the bbox parameter is now optional (and not required as before). If it isn't gived the full result set it returned.
          Hide
          steve Steve Yen added a comment -

          Sounds like it's a U/I level issue at this point.

          Hi Pavel,
          Wondering on difficulty level on fixing this? Need to make the call on whether to defer this to the .next release.

          Thanks,
          Steve

          Show
          steve Steve Yen added a comment - Sounds like it's a U/I level issue at this point. Hi Pavel, Wondering on difficulty level on fixing this? Need to make the call on whether to defer this to the .next release. Thanks, Steve
          Hide
          alkondratenko Aleksey Kondratenko (Inactive) added a comment -

          I'd prefer to defer. My thinking is most of UI code for spatial views is cut&paste from main views code. Doing any significant changes to spatial views code is going to be wasted until we find a proper way to unify this two codes.

          Show
          alkondratenko Aleksey Kondratenko (Inactive) added a comment - I'd prefer to defer. My thinking is most of UI code for spatial views is cut&paste from main views code. Doing any significant changes to spatial views code is going to be wasted until we find a proper way to unify this two codes.
          Hide
          alkondratenko Aleksey Kondratenko (Inactive) added a comment -

          Unless Pavel gives us ready commit sitting in gerrit real soon, I'd strongly suggest abandoning this work

          Show
          alkondratenko Aleksey Kondratenko (Inactive) added a comment - Unless Pavel gives us ready commit sitting in gerrit real soon, I'd strongly suggest abandoning this work
          Hide
          farshid Farshid Ghods (Inactive) added a comment -

          given that spatial indexing may or may not be part 2.0 and that as of today there is no patch on gerrit addressing this issue we are deferring the bug for 2.0.1

          Show
          farshid Farshid Ghods (Inactive) added a comment - given that spatial indexing may or may not be part 2.0 and that as of today there is no patch on gerrit addressing this issue we are deferring the bug for 2.0.1
          Hide
          steve Steve Yen added a comment -

          to 2.0.2 per bug-scrub

          Show
          steve Steve Yen added a comment - to 2.0.2 per bug-scrub
          Hide
          maria Maria McDuff (Inactive) added a comment -

          moving to 2.1. release.

          Show
          maria Maria McDuff (Inactive) added a comment - moving to 2.1. release.
          Hide
          iryna iryna added a comment -

          4.0.0-2093 verified

          Show
          iryna iryna added a comment - 4.0.0-2093 verified

            People

            • Assignee:
              pavel Pavel Blagodov
              Reporter:
              iryna iryna
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Gerrit Reviews

                There are no open Gerrit changes