Skip to content

PCBC-1040: Tracing - include child spans created by the C++ core#249

Merged
avsej merged 5 commits intocouchbase:mainfrom
DemetrisChr:PCBC-1040-core-spans
Apr 1, 2026
Merged

PCBC-1040: Tracing - include child spans created by the C++ core#249
avsej merged 5 commits intocouchbase:mainfrom
DemetrisChr:PCBC-1040-core-spans

Conversation

@DemetrisChr
Copy link
Copy Markdown
Contributor

No description provided.

@DemetrisChr DemetrisChr changed the title PCBC-1040: Tracing - include child spans craeted by the C++ core PCBC-1040: Tracing - include child spans created by the C++ core Mar 20, 2026
@DemetrisChr DemetrisChr marked this pull request as ready for review March 20, 2026 17:46
@avsej avsej force-pushed the PCBC-1040-core-spans branch from b92a4dd to 2f50a3e Compare March 24, 2026 20:43
Comment thread Couchbase/ClusterOptions.php Outdated
Comment thread src/wrapper/connection_handle.cxx Outdated
Comment thread src/wrapper/connection_handle.cxx Outdated
Comment thread src/wrapper/connection_handle.cxx Outdated
Copy link
Copy Markdown
Member

@avsej avsej left a comment

Choose a reason for hiding this comment

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

I think bufferCoreSpans should return false when tracing is disabled.

@DemetrisChr DemetrisChr force-pushed the PCBC-1040-core-spans branch from 2f50a3e to fd35498 Compare March 27, 2026 15:27
@DemetrisChr DemetrisChr requested a review from avsej March 27, 2026 15:29
avsej
avsej previously approved these changes Mar 27, 2026
Copy link
Copy Markdown
Member

@avsej avsej left a comment

Choose a reason for hiding this comment

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

Looks good, but lets merge after #251

@DemetrisChr
Copy link
Copy Markdown
Contributor Author

I've pushed one more patch, adding the couchbase.retries attribute

@DemetrisChr DemetrisChr force-pushed the PCBC-1040-core-spans branch from 112f633 to a8ffc68 Compare March 31, 2026 15:40
avsej
avsej previously approved these changes Mar 31, 2026
@DemetrisChr DemetrisChr force-pushed the PCBC-1040-core-spans branch from 7c32c81 to 8d3a82b Compare April 1, 2026 17:01
@avsej avsej merged commit 9320774 into couchbase:main Apr 1, 2026
291 of 293 checks passed
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