Add async/await support about openwhisk-runtime-dotnet HOT 8 CLOSED

@kamyker are you able to review this PR #25 which adds async support? It would be much appreciated 🙏 if you can take a look.

from openwhisk-runtime-dotnet.

Thanks @kamyker - are you interested in submitting a pull request for this?

from openwhisk-runtime-dotnet.

@rabbah Maybe, any ideas how long it would take IBM to update their image after pr gets accepted? Or maybe I'd be able to make custom docker image and use it instantly instead?

from openwhisk-runtime-dotnet.

You’d need to ask ibm :)

As an Apache project we’d love the contribution, which you could then build and push to dockerhub and use independent of the ibm lifecycle.

from openwhisk-runtime-dotnet.

Tried building and running OpenWhisk but failed as it looks like it may not even work on windows apache/openwhisk-devtools#119 I gave up. Pull itself is trivial but getting test environment seems overwhelming.

from openwhisk-runtime-dotnet.

I'm not sure the best way to go about this, but I believe I can implement it, but is it reasonable to wait until #20 is accepted?

from openwhisk-runtime-dotnet.

I think waiting on merging #20 before working on this new feature is reasonable.

from openwhisk-runtime-dotnet.

Btw if someone needs async before this feature is available (for ex. in IBM) it's possible to instead:
var result = await SomeAsyncFunc();
do:
var result = SomeAsyncFunc().GetAwaiter().GetResult();
Not very efficient as it will use new thread and can prob deadlock in very high usage scenarios but fine in my use case.

from openwhisk-runtime-dotnet.