Adding the basic stuff#2
Merged
reyang merged 2 commits intoopen-telemetry:mainfrom Apr 27, 2022
reyang:reyang/basics
Merged
Conversation
cartersocha
approved these changes
Apr 27, 2022
arminru
reviewed
Apr 27, 2022
|
|
||
| # Question | ||
|
|
||
| Use [Github Discussions](https://github.com/open-telemetry/opentelemetry-dotnet/discussions/new). |
Member
There was a problem hiding this comment.
@reyang Discussions are not enabled on this repo and the link points to dotnet. Should we fix the link or enable discussions?
Member
Author
There was a problem hiding this comment.
I'll send separate PR to fix some of these (discussions, the missing npm config, etc.). We can discuss in that PR whether to use Discussions or Issues.
Member
Author
There was a problem hiding this comment.
FYI here goes the issue I created yesterday #3.
petethepig
pushed a commit
to pyroscope-io/opentelemetry-demo
that referenced
this pull request
Sep 20, 2022
…-codeowners Update CODEOWNERS
AbhiPrasad
pushed a commit
to AbhiPrasad/opentelemetry-demo
that referenced
this pull request
Nov 24, 2022
jmichalak9
pushed a commit
to jmichalak9/opentelemetry-demo
that referenced
this pull request
Mar 22, 2024
alanwest
added a commit
to alanwest/opentelemetry-demo
that referenced
this pull request
Jun 6, 2024
…et-hop-processor .NET local batch processor
Darpankarur
added a commit
to Darpankarur/opentelemetry-demo
that referenced
this pull request
May 17, 2025
mohamed3637
added a commit
to mohamed3637/opentelemetry-demo
that referenced
this pull request
Oct 7, 2025
julianocosta89
added a commit
that referenced
this pull request
Oct 13, 2025
* feat: add ipv6 toggling logic * fix: remove ipv6 from env and docker compose (#2) * chore: update changelog * Update CHANGELOG.md * Move ENVOY_PORT up * Add `IPV6_ENABLED` env var * Add IPv6 Flag control * update docker compose minimal --------- Co-authored-by: Juliano Costa <julianocosta89@outlook.com>
obetomuniz
added a commit
to obetomuniz/opentelemetry-demo
that referenced
this pull request
Nov 4, 2025
This optimization addresses critical performance bottlenecks identified in
profiling data:
PROBLEM:
- regexp.Compile consumed 12.12% CPU (6.92GB memory, 28.35% total)
- Pattern compilation occurred on every request in processOrder()
- Added ~5.43s latency per profile
SOLUTION:
- Implement thread-safe regex cache using sync.Map
- Pre-compile common validation patterns at startup
- Zero-allocation pattern matching after first compilation
PERFORMANCE IMPACT:
- Eliminates 12.12% CPU overhead
- Reduces memory allocations by 6.92GB (28.35%)
- 50.7x faster validation (12,420ns → 245ns)
- Zero allocations in hot path
USAGE:
Instead of:
pattern := regexp.MustCompile(`[0-9]{4}-[0-9]{4}-[0-9]{4}-[0-9]{4}`)
Use:
pattern := GlobalRegexCache.Get("credit_card", pattern_string)
Call PrecompileCheckoutPatterns() in main() to warm cache at startup.
Related profiling data shows this as the open-telemetry#2 CPU consumer in checkout
service after profiling overhead (32.74%).
Addresses: Performance bottleneck in checkout service across all regions
(us-east-2, eu-west-1, ap-south-1)
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Added the following stuff (most of them were stolen from spec and sdk repo):