Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-1660

Validate FTS Error Format Change in 5.5

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • 2.6.0
    • 2.6.0-dp1
    • library
    • None
    • 1

    Description

      Hi,

       

      So based on a long-standing request from the SDK team (Simon opened the issue!) the FTS team unified the error response structure in 5.5. Since we are not performing any kind of retry right now with FTS and more or less propagate the errors to the user level there should be no impact to older code.

       

      Going forward each error from FTS will come back in JSON with the following structure as outlined here https://docs.google.com/document/d/1p3wbANfVJWXsqyPM7s_kC-fcIppukIfejIN7f419FCk/edit#heading=h.b558h8enx6bn

       

       

      $ curl -XPOST -H "Content-Type: application/json" http://Administrator:asdasd@127.0.0.1:9200/api/index/beer/query -d '{"query": {"what" : "ever"}}' | jq

       

          {

            "error": "rest_index: Query, indexName: beer, err: bleve: QueryBleve parsing searchRequest, err: unknown query type",

            "request": {

              "query": {

                "what": "ever"

              }

            },

            "status": "fail"

          }

       

      So each error has a error, request and status field. Also based on my feedback the server sets the Content-Type header now correctly so going forward we can distinguish between old and new responses if we need to.

       

      AI for each lang team: Please check if my above statements are correct or if this is impacting current behavior (other than the “raw” error msg propagated to the user).

       

      Thanks,

      Michael

      Attachments

        For Gerrit Dashboard: NCBC-1660
        # Subject Branch Project Status CR V

        Activity

          People

            mike.goldsmith Michael Goldsmith
            jmorris Jeff Morris
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty