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

Asynchronous call for the dispatcher to avoid blocking request when doing decisions #286

Closed
wants to merge 3 commits into from

Conversation

mdprotacio
Copy link

Summary

  • Dispatcher is supposed to be for asynchronous operation that can be done outside of the normal process like the sending of impression, however, the call to the sending of impression is synchronous and is thus creates a bottleneck in terms of performance. By making the call asynchronous, it improves the performance of decide call.

@mikechu-optimizely
Copy link
Contributor

@mdprotacio Thanks for opening a PR. We'll take a look at your suggestion shortly.

@mdprotacio
Copy link
Author

As issues can be addressed by implementing our own dispatcher, I'm closing this PR.

@mdprotacio mdprotacio closed this Aug 14, 2024
@mikechu-optimizely
Copy link
Contributor

Thanks for the update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants