Skip to content

feat: initial draft non blocking middleware provider#87

Closed
mattsse wants to merge 3 commits intofoundry-rs:masterfrom
mattsse:matt/non-blocking-provider
Closed

feat: initial draft non blocking middleware provider#87
mattsse wants to merge 3 commits intofoundry-rs:masterfrom
mattsse:matt/non-blocking-provider

Conversation

@mattsse
Copy link
Member

@mattsse mattsse commented Oct 4, 2021

Changes

  • Adds a new SyncProvider type that works like a block working of a Middleware, it spawns the actual Middleware to a background thread and executes requests via channels

@mattsse
Copy link
Member Author

mattsse commented Oct 6, 2021

Some initial testing indicates that this is indistinguishable from the BlockingProvider, but perhaps worth giving this another look once more parallel features are implemented

@gakonst
Copy link
Member

gakonst commented Oct 18, 2021

Closing given unclear perf gains as discussed

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.

2 participants