Add option to invoke via async route about connector-sdk HOT 7 CLOSED

Can you check the PR #27 please?

from connector-sdk.

@alexellis What's your preferred way of supporting async function? One possible solution might be add another annotation to function (maybe something like async: true ?)

Is there any suggestion?

from connector-sdk.

cc @martindekov @tessellator

@daikeren I think async_invoke as an env-var would work.

from connector-sdk.

Derek assign: me

from connector-sdk.

I will try to implement this as soon as I can this time this week and will ping 👍

from connector-sdk.

Should I augment this PR to be optional the asynchronous or adding call to the asynchronous gateway path to the connector SDK?

from connector-sdk.

I was thinking of two approaches, amend the asynchronous path to where we configure the gateway URL so now its http://your.ip:port it would become http://your.ip:port/async-function/http://your.ip:port/function/ this would be the easiest approach, the second approach would be to add some sort boolean to the ControllerConfig(I need to make sure thats the most appropriate place). The former would just re-vendor the SDK with this simple change %s/function/%s -> %s/%s and add logic to the implementations of the connector which will build the gateway URL with async/non-async path appended.

from connector-sdk.