Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow non intersecting spans matching in a trace #171

Merged
merged 1 commit into from
Oct 25, 2016

Conversation

RomanDzhabarov
Copy link
Member

Additional data to span (bytes send and received) and matching on x-request-id as guid matching,
guid matching make non intersecting spans assemble as well.

@RomanDzhabarov RomanDzhabarov merged commit 2701313 into master Oct 25, 2016
@RomanDzhabarov RomanDzhabarov deleted the change_to_guid branch October 25, 2016 21:48
PiotrSikora added a commit to PiotrSikora/envoy that referenced this pull request Mar 3, 2020
…y#171)

Description: Remove empty slices off the end of buffers after calls to OwnedImpl::commit. The slices reserved when OwnedImpl::reserve is called will sit unused in cases where the 0 bytes are commited, for example, when socket read returns 0 bytes EAGAIN. Trapped slices act like a memory leak until there is a successful read or the socket is closed.
Risk Level: low
Testing: unit
Docs Changes: n/a
Release Notes: n/a

Signed-off-by: Asra Ali <asraa@google.com>

Co-authored-by: asraa <asraa@google.com>
wolfguoliang pushed a commit to wolfguoliang/envoy that referenced this pull request Jan 23, 2021
zh-translation: /intro/arch_overview/listeners/listeners.rst
jpsim pushed a commit that referenced this pull request Nov 28, 2022
Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this pull request Nov 29, 2022
Signed-off-by: Mike Schore <mike.schore@gmail.com>
Signed-off-by: JP Simard <jp@jpsim.com>
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.

None yet

2 participants