add APIs to execution context for middleware support - #9
add APIs to execution context for middleware support#9kaibocai (kaibocai) wants to merge 6 commits into
Conversation
|
kaibocai (@kaibocai) If you add support for the ExecutionContext methods to the Java worker, I can test them out in the Quarkus extension I wrote to integration with azure functions. |
|
|
||
| void setMiddlewareOutput(Object value); | ||
|
|
||
| void setFunctionInstance(Object functionInstance); |
There was a problem hiding this comment.
Might consider moving these middleware specific methods like setFunctionInstance to a separate interface that extends extends ExecutionContext. ExecutionContext is available to the application developer and these methods might not be something you want exposed to them. See my PR here which shows this:
There was a problem hiding this comment.
Thank you for your valuable suggestion! We will discuss it with the team.
Hi Bill Burke (@patriot1burke) , please use this java worker and this pr core library for local testing.
|
|
Refer to #10 (comment). |
add APIs to execution context for middleware support