-
Notifications
You must be signed in to change notification settings - Fork 692
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
feat(instrumentation): Add event-based tracking implementation across providers #2541
base: main
Are you sure you want to change the base?
feat(instrumentation): Add event-based tracking implementation across providers #2541
Conversation
…raceloop#2456) - Add event-based tracking support while maintaining legacy behavior - Implement use_legacy_attributes config parameter - Add event logger integration with proper trace context - Add streaming support with event buffering - Include comprehensive test coverage - Add sample app integration Part of the larger effort to support event-based tracking across all packages. Signed-off-by: David Anyatonwu <[email protected]>
…ventions Signed-off-by: David Anyatonwu <[email protected]>
@nirga Could i get a review, to confirm I am on the right track. Thanks. |
@onyedikachi-david can you rebase so we can run the CI? |
alright |
Done @nirga |
@onyedikachi-david looks like your tests are failing - can you take a look? |
Okay |
Hi, @nirga Hope you don't mind I open another PR, the conflicts are a lot. I think it will be more cleaner. Thanks. What do you think? |
Sure @onyedikachi-david |
Thank you. On it. |
/claim #2456
Fixes: #2456
feat(instrumentation): ...
orfix(instrumentation): ...
.Description
This PR adds event-based tracking implementation across multiple providers while maintaining backward compatibility with the legacy attribute-based approach.
Completed Implementations:
Pending Implementations:
Key Changes:
use_legacy_attributes
supportEach provider implementation includes:
events.py
)config.py
)__init__.py
)test_events.py
)