Skip to content

CXXCBC-381: use shared_ptr for transactions_context#590

Merged
avsej merged 3 commits intocouchbase:mainfrom
avsej:CXXCBC-381-shared_ptr-for-attempt_context
Jun 17, 2024
Merged

CXXCBC-381: use shared_ptr for transactions_context#590
avsej merged 3 commits intocouchbase:mainfrom
avsej:CXXCBC-381-shared_ptr-for-attempt_context

Conversation

@avsej
Copy link
Copy Markdown
Member

@avsej avsej commented Jun 13, 2024

No description provided.

@avsej avsej requested a review from a team June 13, 2024 18:56
@avsej avsej force-pushed the CXXCBC-381-shared_ptr-for-attempt_context branch 2 times, most recently from 6bdba5f to d9a7398 Compare June 14, 2024 00:55
@DemetrisChr
Copy link
Copy Markdown
Contributor

We might need to define a feature macro for this so we can keep compatibility with previous versions in the FIT performer

@avsej avsej force-pushed the CXXCBC-381-shared_ptr-for-attempt_context branch from d9a7398 to 7373337 Compare June 14, 2024 19:30
@avsej avsej force-pushed the CXXCBC-381-shared_ptr-for-attempt_context branch from 7373337 to 1039159 Compare June 14, 2024 21:46
@avsej avsej force-pushed the CXXCBC-381-shared_ptr-for-attempt_context branch from b10b9df to 70aa753 Compare June 17, 2024 04:14
@avsej avsej merged commit df0bc2d into couchbase:main Jun 17, 2024
@avsej avsej deleted the CXXCBC-381-shared_ptr-for-attempt_context branch June 17, 2024 05:15
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