-
Notifications
You must be signed in to change notification settings - Fork 327
Feature/async #61
base: master
Are you sure you want to change the base?
Feature/async #61
Conversation
Hi @turowicz, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution! TTYL, AZPRBOT; |
Related Issue |
Signed the document |
We've been extensively testing my code change and so far it works great in production, average CPU usage went down from 90% to 4% on exchange operations. |
@turowicz, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR. |
I also looking into async a while ago. I did take a a bit of a different approache. I decided to create a new project targeting .NET 4.5 to leave the existing code in place and use partials to extend the classes with Async methods (not everything has support nor has it been properly tested). I cleaned/updated the code some days ago to feed as input for a discussion on async for EWS-Managed and whether it is desired to have it included (e.g. the framework is not threadsafe to begin with). The branch can be found at async. |
@WGroenestein good approach - I honestly don't mind which one is chosen, as long as we can use async within EWS. |
…ice.cs to current master version. (only moved items that prevented compile) OfficeDev#61
Any update on a merge for the async support ? or a pointer to an existing Async EWS Managed API somewhere else ? |
Like @ticapix said is there any update on this? This is a necessity nowadays to have async methods. |
I already made a separate project with everything async. Please check here |
Oh, very nice! Did you base it on HttpClient? |
Introduce possibility of making true async-await requests for a subset of basic methods.