Skip to content

CXXCBC-499 Use couchbase::error for Query, Search and Analytics operations#557

Merged
Matt-Woz merged 14 commits intocouchbase:CXXCBC-496-errorfrom
Matt-Woz:CXXCBC-499_error_query_search_analytics
Apr 15, 2024
Merged

CXXCBC-499 Use couchbase::error for Query, Search and Analytics operations#557
Matt-Woz merged 14 commits intocouchbase:CXXCBC-496-errorfrom
Matt-Woz:CXXCBC-499_error_query_search_analytics

Conversation

@Matt-Woz
Copy link
Copy Markdown
Contributor

Changes:

  • Convert query, search & analytics to use couchbase::error
  • Add assign & as() methods for JSON conversion for analytics, search, query (and http) error contexts
  • Add make_error util methods
  • Add as() method to error context

@Matt-Woz Matt-Woz requested a review from DemetrisChr April 11, 2024 11:57
Comment thread couchbase/retry_reason.hxx Outdated
Comment thread core/impl/error.hxx Outdated
Comment thread couchbase/query_options.hxx Outdated
Comment thread core/impl/error.cxx Outdated
@Matt-Woz Matt-Woz requested a review from DemetrisChr April 11, 2024 15:23
Comment thread core/impl/cluster.cxx Outdated
Comment thread couchbase/retry_reason.hxx Outdated
@Matt-Woz Matt-Woz merged commit 5392f1c into couchbase:CXXCBC-496-error Apr 15, 2024
@Matt-Woz Matt-Woz deleted the CXXCBC-499_error_query_search_analytics branch April 15, 2024 15:59
avsej added a commit that referenced this pull request May 27, 2024
…rror (#570)

* CXXCBC-499 Use couchbase::error for Query, Search and Analytics operations (#557)

* CXXCBC-492: Use couchbase::error for KV APIs (#562)

* CXXCBC-498 Use couchbase:error for management operations (#560)

* CXXCBC-500 Use couchbase::error for transactions (#564)

* CXXCBC-508 Change connect, ping, diagnostics to use couchbase::error (#567)

---------
Co-authored-by: Mateusz <[email protected]>
Co-authored-by: Dimitris Christodoulou <[email protected]>
Co-authored-by: Sergey Avseyev <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants