Couchbase .NET client library

error format changing

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Duplicate
  • Affects Version/s: None
  • Fix Version/s: 1.2.1
  • Component/s: library
  • Labels:
    None

Description

The error format is changing, need to ensure that's updated in the client.

Rough example format:

{"total_rows":0,"rows":[

],
"errors": [{"from":"127.0.0.1:5984","reason":"Design document `_design/testfoobar` missing in database `test_db_b`."},
{"from":"http://localhost:5984/_view_merge/","reason":"Design document `_design/testfoobar` missing in database `test_db_c`."}]}

Activity

Descending order - Click to sort in ascending order
Hide
Permalink
John Zablocki added a comment -
This error format is out of date. Current error format is below. As part of NCBC-165, error handling was cleaned up.

{"error":"error","reason":"Error opening view `does_not_exist`, from set `default`, design document `_design/cities`: {not_found,\n missing_named_view}"}

http://localhost:8092/default/_design/cities/_view/does_not_exist
Show
John Zablocki added a comment - This error format is out of date. Current error format is below. As part of NCBC-165 , error handling was cleaned up. {"error":"error","reason":"Error opening view `does_not_exist`, from set `default`, design document `_design/cities`: {not_found,\n missing_named_view}"} http://localhost:8092/default/_design/cities/_view/does_not_exist
Hide
Permalink
John Zablocki added a comment -
Code is handling errors, but need unit test.
Show
John Zablocki added a comment - Code is handling errors, but need unit test.
Hide
Permalink
Matt Ingenthron added a comment -
The code needs to be inspected to verify this error handling.
Show
Matt Ingenthron added a comment - The code needs to be inspected to verify this error handling.
Hide
Permalink
Matt Ingenthron added a comment -
Was this handled in 0.9?
Show
Matt Ingenthron added a comment - Was this handled in 0.9?
Hide
Permalink
Matt Ingenthron added a comment -
Here's a better example of the format. It changed slightly again.

{"total_rows":0,"rows":[
],
"errors":[
{"from":"local","reason":"Design document `_design/testfoobar` missing
in database `test_db_b`."},
{"from":"http://localhost:5984/_view_merge/","reason":"Design document
`_design/testfoobar` missing in database `test_db_c`."}
]}
Show
Matt Ingenthron added a comment - Here's a better example of the format. It changed slightly again. {"total_rows":0,"rows":[ ], "errors":[ {"from":"local","reason":"Design document `_design/testfoobar` missing in database `test_db_b`."}, {"from":" http://localhost:5984/_view_merge/ ","reason":"Design document `_design/testfoobar` missing in database `test_db_c`."} ]}

People

  • Assignee:
    John Zablocki
    Reporter:
    Matt Ingenthron
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: