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

Views UI broken for binary documents

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 5.5.0, 5.5.1, 5.5.2
    • 5.5.3, 6.0.1
    • UI

    Description

      1. Create default bucket - load it with binary documents
      2. Create a design doc 'dev_test' and a view 'test' under it
      3. Click on the view in UI
      4. Results page does not show any documents or results - we just see ajax spinner going on forever - 404 errors seen on inspecting the page in the browser

      Issue does not happen while dealing with json documents - results are shown in UI as expected

      REST API works fine for both json and binary docs

      Works in 5.1.3 - broken since 5.5.0

      Attachments

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

        Activity

          Jeelan Poola please triage this and see if this needs to be fixed in 5.5.3

          arunkumar Arunkumar Senthilnathan added a comment - Jeelan Poola please triage this and see if this needs to be fixed in 5.5.3
          dfinlay Dave Finlay added a comment -

          Timofey - can you take a look? Is this related to the changes we made in the docs REST API?

          dfinlay Dave Finlay added a comment - Timofey - can you take a look? Is this related to the changes we made in the docs REST API?

          Dave Finlay

          I reproduced it. Doesn't look like a server issue. No signs of errors in server logs. I also don't see any errors in browser's console either. Arunkumar Senthilnathan, these errors in console were probably generated while you were creating views and documents and I suspect they are not related to the issue. 

          timofey.barmin Timofey Barmin added a comment - Dave Finlay I reproduced it. Doesn't look like a server issue. No signs of errors in server logs. I also don't see any errors in browser's console either. Arunkumar Senthilnathan , these errors in console were probably generated while you were creating views and documents and I suspect they are not related to the issue. 
          dfinlay Dave Finlay added a comment -

          Thanks Timofey. Assigning back to views for now.

          dfinlay Dave Finlay added a comment - Thanks Timofey. Assigning back to views for now.
          ankit.prabhu Ankit Prabhu added a comment -

          Pavel Blagodov Dave Finlay

          js_beautify function is unable to process "undefined" value which is given in doc.json when doc is binary. This lead view ui to show the ajax spinner.

          https://github.com/couchbase/ns_server/blob/6e2fc19991bc1432cb1e4d787bd1547318887abe/priv/public/ui/app/mn_admin/mn_indexes/mn_views/editing/mn_views_editing_service.js#L105

          ankit.prabhu Ankit Prabhu added a comment - Pavel Blagodov Dave Finlay js_beautify function is unable to process "undefined" value which is given in doc.json when doc is binary. This lead view ui to show the ajax spinner. https://github.com/couchbase/ns_server/blob/6e2fc19991bc1432cb1e4d787bd1547318887abe/priv/public/ui/app/mn_admin/mn_indexes/mn_views/editing/mn_views_editing_service.js#L105

          Dave Finlay Mike Wiederhold [X] Do we need to have this fix in 5.5.3 ?

          mihir.kamdar Mihir Kamdar (Inactive) added a comment - Dave Finlay Mike Wiederhold [X] Do we need to have this fix in 5.5.3 ?

          Build couchbase-server-5.5.3-4038 contains ns_server commit 83c5d3a with commit message:
          MB-32142: Pass empty string instead of undefined to js_beautify func

          build-team Couchbase Build Team added a comment - Build couchbase-server-5.5.3-4038 contains ns_server commit 83c5d3a with commit message: MB-32142 : Pass empty string instead of undefined to js_beautify func
          wayne Wayne Siu added a comment -

          Pavel Blagodov Mike Wiederhold [X]

          is the change also in alice branch yet?

          wayne Wayne Siu added a comment - Pavel Blagodov Mike Wiederhold [X] is the change also in alice branch yet?

          Build couchbase-server-6.0.1-1985 contains ns_server commit fb8e6ad with commit message:
          Merge remote-tracking branch 'couchbase/vulcan' into alice

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.1-1985 contains ns_server commit fb8e6ad with commit message: Merge remote-tracking branch 'couchbase/vulcan' into alice

          Build couchbase-server-6.0.1-1985 contains ns_server commit 83c5d3a with commit message:
          MB-32142: Pass empty string instead of undefined to js_beautify func

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.1-1985 contains ns_server commit 83c5d3a with commit message: MB-32142 : Pass empty string instead of undefined to js_beautify func

          Verified in 5.5.3-4038

          arunkumar Arunkumar Senthilnathan added a comment - Verified in 5.5.3-4038

          Verified in 6.0.1-1985

          arunkumar Arunkumar Senthilnathan added a comment - Verified in 6.0.1-1985

          Build couchbase-server-6.5.0-1846 contains ns_server commit aec78a9 with commit message:
          Merge remote-tracking branch 'couchbase/alice'.

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1846 contains ns_server commit aec78a9 with commit message: Merge remote-tracking branch 'couchbase/alice'.

          Build couchbase-server-6.5.0-1846 contains ns_server commit fb8e6ad with commit message:
          Merge remote-tracking branch 'couchbase/vulcan' into alice

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1846 contains ns_server commit fb8e6ad with commit message: Merge remote-tracking branch 'couchbase/vulcan' into alice

          Build couchbase-server-6.5.0-1846 contains ns_server commit 83c5d3a with commit message:
          MB-32142: Pass empty string instead of undefined to js_beautify func

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1846 contains ns_server commit 83c5d3a with commit message: MB-32142 : Pass empty string instead of undefined to js_beautify func

          People

            pavel Pavel Blagodov
            arunkumar Arunkumar Senthilnathan
            Votes:
            0 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty