Skip to content

CXXCBC-500 Use couchbase::error for transactions#564

Merged
Matt-Woz merged 15 commits intocouchbase:CXXCBC-496-errorfrom
Matt-Woz:transactions_error_changes
May 16, 2024
Merged

CXXCBC-500 Use couchbase::error for transactions#564
Matt-Woz merged 15 commits intocouchbase:CXXCBC-496-errorfrom
Matt-Woz:transactions_error_changes

Conversation

@Matt-Woz
Copy link
Copy Markdown
Contributor

Changes:

  • Changed transactions APIs to use couchbase::error
  • Made couchbase::error polymorphic & extended by transaction_operation_failed

@Matt-Woz Matt-Woz requested a review from DemetrisChr April 26, 2024 11:57
Comment thread test/test_transaction_public_blocking_api.cxx
Comment thread core/error_context/query_public_json.hxx Outdated
@avsej
Copy link
Copy Markdown
Member

avsej commented May 6, 2024

@Matt-Woz, @DemetrisChr is this PR still in progress?

Comment thread core/transactions/attempt_context_impl.cxx
@Matt-Woz Matt-Woz merged commit 6b7636a into couchbase:CXXCBC-496-error May 16, 2024
@Matt-Woz Matt-Woz deleted the transactions_error_changes branch May 16, 2024 09:27
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.

3 participants