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

User is able to save uncompilable map function for spatial view

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0
    • Component/s: None
    • Security Level: Public
    • Labels:
      None
    • Environment:

      Description

      putting
      {"views":{"df":{"map":"function (doc, meta)

      {\n emit(meta.id, null);\n}

      "}},"spatial":{"ff":"function (doc) {\n if (doc.geometry) {\n emit(doc.geometry, null);\n\n"}} (2 curlies are missing in map fn)
      do not return any error

      response is

      {"ok":true,"id":"_design/dev_d"}
      No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

        iryna iryna created issue -
        Hide
        farshid Farshid Ghods (Inactive) added a comment -

        Volker,

        please assign the ticket back to Alk if its UI related

        Show
        farshid Farshid Ghods (Inactive) added a comment - Volker, please assign the ticket back to Alk if its UI related
        farshid Farshid Ghods (Inactive) made changes -
        Field Original Value New Value
        Fix Version/s 2.0 [ 10114 ]
        Assignee Filipe Manana [ filipemanana ] Volker Mische [ vmx ]
        Hide
        FilipeManana Filipe Manana (Inactive) added a comment -

        Yep. This is exactly what used to happen with the mapreduce engine and old couchdb view engine.
        We haven't done many things for the geocouch engine, and until it's more unified with the other view engine, it will likely remain like that.

        That will be post 2.0.

        Show
        FilipeManana Filipe Manana (Inactive) added a comment - Yep. This is exactly what used to happen with the mapreduce engine and old couchdb view engine. We haven't done many things for the geocouch engine, and until it's more unified with the other view engine, it will likely remain like that. That will be post 2.0.
        iryna iryna made changes -
        Summary User is able to save uncomilable map function for spatial view User is able to save uncompilable map function for spatial view
        Hide
        vmx Volker Mische added a comment -

        I'll try to fix it with adding the validation.

        Show
        vmx Volker Mische added a comment - I'll try to fix it with adding the validation.
        Hide
        steve Steve Yen added a comment -

        changing priority to major

        Show
        steve Steve Yen added a comment - changing priority to major
        steve Steve Yen made changes -
        Priority Minor [ 4 ] Major [ 3 ]
        Show
        vmx Volker Mische added a comment - Ready for review: http://review.couchbase.org/21890 http://review.couchbase.org/21891
        Hide
        vmx Volker Mische added a comment -

        Merged!

        Show
        vmx Volker Mische added a comment - Merged!
        vmx Volker Mische made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in github-couchdb-preview #524 (See http://qa.hq.northscale.net/job/github-couchdb-preview/524/)
        MB-6990: Add validation of the spatial function (Revision ebe5a3ac0b35ae5c6d75954836b1ff2ddbfb34a5)

        Result = SUCCESS
        Farshid Ghods :
        Files :

        • src/couchdb/couch_db.erl
        Show
        thuan Thuan Nguyen added a comment - Integrated in github-couchdb-preview #524 (See http://qa.hq.northscale.net/job/github-couchdb-preview/524/ ) MB-6990 : Add validation of the spatial function (Revision ebe5a3ac0b35ae5c6d75954836b1ff2ddbfb34a5) Result = SUCCESS Farshid Ghods : Files : src/couchdb/couch_db.erl
        Hide
        thuan Thuan Nguyen added a comment -

        Integrated in github-couchdb-preview #525 (See http://qa.hq.northscale.net/job/github-couchdb-preview/525/)
        MB-6990 Ignore missing module during make check (Revision ab2b0c230bffb74fcb3771db5871322c8ac806d0)

        Result = SUCCESS
        steve :
        Files :

        • src/couchdb/couch_db.erl
        Show
        thuan Thuan Nguyen added a comment - Integrated in github-couchdb-preview #525 (See http://qa.hq.northscale.net/job/github-couchdb-preview/525/ ) MB-6990 Ignore missing module during make check (Revision ab2b0c230bffb74fcb3771db5871322c8ac806d0) Result = SUCCESS steve : Files : src/couchdb/couch_db.erl
        Hide
        iryna iryna added a comment -

        verified on 1901
        MB-7029 opened

        Show
        iryna iryna added a comment - verified on 1901 MB-7029 opened
        iryna iryna made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        kzeller kzeller added a comment -

        Added to RN: For geo/spatial indexes, which is an experimental
        feature in Couchbase 2.0, we now provide validation
        of spatial functions.

        Show
        kzeller kzeller added a comment - Added to RN: For geo/spatial indexes, which is an experimental feature in Couchbase 2.0, we now provide validation of spatial functions.

          People

          • Assignee:
            vmx Volker Mische
            Reporter:
            iryna iryna
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes