Skip to content

CXXCBC-510: Support binary objects in transactions#576

Merged
avsej merged 1 commit intocouchbase:mainfrom
avsej:CXXCBC-510-binary-transactions
Jun 11, 2024
Merged

CXXCBC-510: Support binary objects in transactions#576
avsej merged 1 commit intocouchbase:mainfrom
avsej:CXXCBC-510-binary-transactions

Conversation

@avsej
Copy link
Copy Markdown
Member

@avsej avsej commented Jun 5, 2024

No description provided.

@avsej avsej requested review from DemetrisChr and Matt-Woz June 5, 2024 06:56
@avsej avsej marked this pull request as draft June 5, 2024 06:56
@avsej
Copy link
Copy Markdown
Member Author

avsej commented Jun 5, 2024

Several FIT tests still failing, in particular

  • ExtBinarySupportCleanupTest
  • ExtBinarySupportTranscoderWritingTest

@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch 2 times, most recently from d7a2477 to b18f470 Compare June 6, 2024 05:26
@avsej avsej marked this pull request as ready for review June 6, 2024 05:26
@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch 2 times, most recently from 637b7da to 0b24861 Compare June 6, 2024 05:35
@avsej
Copy link
Copy Markdown
Member Author

avsej commented Jun 6, 2024

Right now tests fail to compile, this is due to the main test coverage is done by FIT. I'm going to update unit tests tomorrow. But the implementation is complete now.

@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch 17 times, most recently from e40f39d to 06a5f18 Compare June 10, 2024 05:44
@avsej avsej requested a review from a team June 11, 2024 00:35
@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch from 25f4a27 to 767300e Compare June 11, 2024 00:45
Copy link
Copy Markdown
Member

@brett19 brett19 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests should be fixed before merging.

Comment thread .github/workflows/windows.yml Outdated
@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch 2 times, most recently from 145e0b7 to bb8b875 Compare June 11, 2024 20:55
brett19
brett19 previously approved these changes Jun 11, 2024
@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch from bb8b875 to 5b485c0 Compare June 11, 2024 22:10
@avsej avsej force-pushed the CXXCBC-510-binary-transactions branch from 5b485c0 to 15c488d Compare June 11, 2024 22:36
@avsej avsej merged commit 1244f3b into couchbase:main Jun 11, 2024
@avsej avsej deleted the CXXCBC-510-binary-transactions branch June 11, 2024 23:05
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