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

Query errors when design document names have a / (forward slash) character as part of their name

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-beta-2
    • Fix Version/s: 2.0.1
    • Component/s: ns_server, view-engine
    • Security Level: Public
    • Labels:
      None

      Description

      But i have 1. view which doesn't work ->
      from: http://192.168.42.18:8092/_view_merge/?group=true&limit=10
      reason:

      {<<"bad_request">>, <<"A `view` property must have the shape `ddoc_name/view_name`.">>}

      Well i think is not a big deal i can create a new design document but i was wondering.

      Btw. here is the log file of the rebalancing i read couple of warning's and error's:
      https://s3-eu-west-1.amazonaws.com/dl-crowdpark/couchbase/logs/ns-diag-20121114133046.txt.zip

      # Subject Project Status CR V
      For Gerrit Dashboard: &For+MB-7193=message:MB-7193

        Activity

        Hide
        steve Steve Yen added a comment -

        moved per bug-scrub

        Show
        steve Steve Yen added a comment - moved per bug-scrub
        Hide
        vmx Volker Mische added a comment -

        Here are the commits:
        http://review.couchbase.org/22567
        http://review.couchbase.org/22568
        http://review.couchbase.org/22569

        WARNING: I haven't run any automated tests yet. I'll do tomorrow in case no one steps in and does it in the meanwhile.

        Show
        vmx Volker Mische added a comment - Here are the commits: http://review.couchbase.org/22567 http://review.couchbase.org/22568 http://review.couchbase.org/22569 WARNING : I haven't run any automated tests yet. I'll do tomorrow in case no one steps in and does it in the meanwhile.
        Hide
        vmx Volker Mische added a comment -

        I've run the following testrunner tests on those commits:
        make simple-test
        conf/py-view-pre-merge.conf
        conf/py-spatial.conf
        conf/py-spatialquery.conf

        As always, I never got them all pass in one run, but after several runs and commenting out some tests (and running them locally and on the VM) I got every test pass eventually once.

        One exception is spatialviewtests.SpatialViewTests.test_spatial_view_on_x_docs_y_design_docs,num-docs=10000,num-design-docs=10 which only worked with a lower number of design documents (5 worked on my machine, with 10 some nodes went down).

        So those commits are verified enough for my taste and ready to go into 2.0.2

        Show
        vmx Volker Mische added a comment - I've run the following testrunner tests on those commits: make simple-test conf/py-view-pre-merge.conf conf/py-spatial.conf conf/py-spatialquery.conf As always, I never got them all pass in one run, but after several runs and commenting out some tests (and running them locally and on the VM) I got every test pass eventually once. One exception is spatialviewtests.SpatialViewTests.test_spatial_view_on_x_docs_y_design_docs,num-docs=10000,num-design-docs=10 which only worked with a lower number of design documents (5 worked on my machine, with 10 some nodes went down). So those commits are verified enough for my taste and ready to go into 2.0.2
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in github-couchdb-preview #540 (See http://qa.hq.northscale.net/job/github-couchdb-preview/540/)
        MB-7193: Make MapReduce Views work that contain slashes (Revision 77bd011cdf89ad84c6e2f82aae665d58a3e028bb)

        Result = SUCCESS
        Filipe David Borba Manana :
        Files :

        • src/couch_index_merger/src/couch_httpd_view_merger.erl
        • src/couch_index_merger/src/couch_view_merger.erl
        Show
        thuan Thuan Nguyen added a comment - Integrated in github-couchdb-preview #540 (See http://qa.hq.northscale.net/job/github-couchdb-preview/540/ ) MB-7193 : Make MapReduce Views work that contain slashes (Revision 77bd011cdf89ad84c6e2f82aae665d58a3e028bb) Result = SUCCESS Filipe David Borba Manana : Files : src/couch_index_merger/src/couch_httpd_view_merger.erl src/couch_index_merger/src/couch_view_merger.erl
        Hide
        FilipeManana Filipe Manana (Inactive) added a comment -

        Merged.

        Show
        FilipeManana Filipe Manana (Inactive) added a comment - Merged.

          People

          • Assignee:
            vmx Volker Mische
            Reporter:
            farshid Farshid Ghods (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: