Uploaded image for project: 'Couchbase Documentation'
  1. Couchbase Documentation
  2. DOC-7979

[DOC] ERROR Messages inconsistency

    XMLWordPrintable

Details

    • DOC-2021-Feb28-S4, DOC-2021-Mar14-S5, DOC-2021-Mar28-S6, DOC-2021-Apr11-S7, DOC-2021-Apr25-S8, DOC-2021-May09-S9, DOC-2021-May23-S10, DOC-2021-Jun06-S11, DOC-2021-Jun20-S12, DOC-2021-Jul04-S13, DOC-2021-Jul18-S14, DOC-2021-Aug01-S15, DOC-2021-Aug15-S16, DOC-2021-Aug29-S17, DOC-2021-Sep12-S18, DOC-2021-Sep26-S19, DOC-2021-Oct10-S20, DOC-2021-Oct24-S21, DOC-2021-Nov07-S22, DOC-2021-Nov21-S23, DOC-2021-Dec05-S24, DOC-2021-Dec19-S25, DOC-2021-Dec31-S26, DOC-2022-S1, DOC-2022-S2

    Description

      First, info on this page (https://docs.couchbase.com/nodejs-sdk/current/ref/error-codes.html) are not quite consistent with error messages of codes (https://docs.couchbase.com/nodejs-sdk/current/howtos/error-handling.html)

      Should we expect couchbaseError cause 105 or cause

      {kv!code0x02}

      or instanceof couchbase.DocumentExistsError (which seems behaviour of code)?

      Second, "cluster was closed" and "parent cluster has been closed" errors that code can throw are not documented.

      Attachments

        Issue Links

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

          Activity

            I do get `[Error: LCB_ERR_CONNECTION_REFUSED (1024)` errors e.g. if database isn't open, that aren't an instance of CouchbaseError, so that feels like a bug, checking with sdk devs.

            hakim.cassimally Hakim Cassimally added a comment - I do get `[Error: LCB_ERR_CONNECTION_REFUSED (1024)` errors e.g. if database isn't open, that aren't an instance of CouchbaseError, so that feels like a bug, checking with sdk devs.

            modules/shared/partials/error-ref.adoc:The base exception, CouchbaseException

            … is called CouchbaseError in Node SDK
             
            In general, there seems to be inconsistency between the 2 terms.
            (As well as the duplicate/wrong ones below, there may be others like NodeJS that include a partial referring to the wrong base class)
             
            ❯ rg -l CouchbaseException | cut -f1 -d/ | uniq
            couchbase-net-client
            docs-sdk-python
            docs-sdk-nodejs (WRONG)
            docs-sdk-scala
            couchbase-python-client
            php-couchbase
            spring-data-couchbase
            docs-sdk-java
            docs-sdk-scala
            docs-sdk-common
            couchbase-jvm-clients
            docs-sdk-dotnet
            docs-site❯ rg -l CouchbaseError | cut -f1 -d/ | uniq
            couchnode
            couchbase-ruby-client
            docs-sdk-ruby
            couchbase-rs
            node-ottoman
            docs-sdk-python
            docs-sdk-dotnet

            hakim.cassimally Hakim Cassimally added a comment - modules/shared/partials/error-ref.adoc:The base exception,  CouchbaseException … is called  CouchbaseError  in Node SDK   In general, there seems to be inconsistency between the 2 terms. (As well as the duplicate/wrong ones below, there may be others like NodeJS that include a partial referring to the wrong base class)   ❯ rg -l CouchbaseException | cut -f1 -d/ | uniq couchbase-net-client docs-sdk-python docs-sdk-nodejs (WRONG) docs-sdk-scala couchbase-python-client php-couchbase spring-data-couchbase docs-sdk-java docs-sdk-scala docs-sdk-common couchbase-jvm-clients docs-sdk-dotnet docs-site❯ rg -l CouchbaseError | cut -f1 -d/ | uniq couchnode couchbase-ruby-client docs-sdk-ruby couchbase-rs node-ottoman docs-sdk-python docs-sdk-dotnet
            hakim.cassimally Hakim Cassimally added a comment - More shared PR  https://github.com/couchbase/docs-sdk-common/pull/39
            hakim.cassimally Hakim Cassimally added a comment - Node PR (depends on above)  https://github.com/couchbase/docs-sdk-nodejs/pull/156

            closed shared PR, as that's overcomplicated.

            Bulk of work was done on this ticket (see Node PR above)

            hakim.cassimally Hakim Cassimally added a comment - closed shared PR, as that's overcomplicated. Bulk of work was done on this ticket (see Node PR above)

            People

              hakim.cassimally Hakim Cassimally
              docsite-collector Docs Site Collector
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty