-
Notifications
You must be signed in to change notification settings - Fork 103
(otelarrowreceiver): asynchronous stream operations #181
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
Merged
jmacd
merged 29 commits into
open-telemetry:main
from
moh-osman3:mohosman/apply-bounded-semaphore
May 9, 2024
Merged
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
9a73c2a
add semaphore files
moh-osman3 e30c220
add uncompSz to proto defintion and apply semaphore
moh-osman3 3ec41e4
add AdmissionLimitMiB separate from MemoryLimitMiB, always record unc…
moh-osman3 8e16255
closer to working
moh-osman3 2a3fa37
return error in goroutine
moh-osman3 45f2f9a
refactor solve race, broken tests still
moh-osman3 3e68d50
almost
moh-osman3 212fca9
painful but fixed?
moh-osman3 0d9f180
rm
moh-osman3 366936e
rebase and fix tests again
moh-osman3 4c84297
fix race
moh-osman3 e1b920a
remove unneded package
moh-osman3 f93a37c
remove replace
moh-osman3 77364fe
review feedback
moh-osman3 a8448e4
remove unneeded arg
moh-osman3 12f1b8c
rm uncompressed_size from proto
moh-osman3 6f8ba19
add header, acquire diff
moh-osman3 af29366
fix potential deadlock
moh-osman3 3b09b6c
unit tests
moh-osman3 1258605
use assertEqualUnsortedSpans
moh-osman3 25366e0
remove unneeded helper functions and use otelAssert.Equiv
moh-osman3 40f7fa9
update changelog
moh-osman3 7db9a40
improve readability with response.bytesToRelease
moh-osman3 8d45b90
remove deprecated
moh-osman3 8ea4949
gofmt
moh-osman3 aacba4e
rm newline
moh-osman3 b97b100
go mod tidy
moh-osman3 e9e2d31
add client address if available
moh-osman3 985ab4b
Merge branch 'main' into mohosman/apply-bounded-semaphore
jmacd File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -49,4 +49,4 @@ service: | |
| address: 127.0.0.1:8888 | ||
| level: detailed | ||
| logs: | ||
| level: info | ||
| level: info | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -57,4 +57,4 @@ service: | |
| telemetry: | ||
| metrics: | ||
| address: 127.0.0.1:8889 | ||
| level: normal | ||
| level: normal | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.