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

feat(core): clean up providers during shutdown and enable transaction context propagation #22

Open
wants to merge 17 commits into
base: qa
Choose a base branch
from

Conversation

ubabar123
Copy link

@ubabar123 ubabar123 commented Dec 18, 2024

This PR

  • Adds graceful provider cleanup during application shutdown.
  • Implements transaction context propagation for evaluation contexts such as HTTP requests or threads.

Notes

  • Provider shutdown ensures that resources are cleaned up properly when the application stops.
  • Transaction context propagation improves context management for multi-threaded or HTTP request-based workloads.

Follow-up Tasks

  • Add more testing coverage for edge cases in context propagation.
  • Update documentation to reflect these changes.

How to Test

  1. Start the application and initialize multiple providers.
  2. Trigger application shutdown (e.g., stop the server).
  3. Observe if providers are cleaned up without errors.
  4. Verify context propagation works in scenarios like HTTP requests or multi-threaded tasks.

ubabar123 and others added 8 commits December 11, 2024 14:29
Signed-off-by: Usman <[email protected]>
…re#18)

## This PR

- Implements enhanced provider extensibility framework
- Adds comprehensive plugin architecture for provider extensions
- Implements provider configuration and authentication systems

### Follow-up Tasks
-add test files (hooks, feature client and API)

Signed-off-by: Brian Chebon <[email protected]>
…transaction context propagation

- Add graceful provider cleanup during application shutdown.
- Implement transaction context propagation for setting evaluation contexts, such as HTTP requests or threads.

Signed-off-by: Usman <[email protected]>
@ABC2015
Copy link

ABC2015 commented Dec 18, 2024

@ubabar123 thanks for the pr request - please resolve the pending conflicts so we can review.

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.

3 participants