middleware support logics #659

kaibocai · 2022-09-23T03:04:43Z

Issue describing the changes in this PR

resolves #issue_for_this_pr

Pull request checklist

My changes do not require documentation changes
- Otherwise: Documentation issue linked to PR
My changes should not be added to the release notes for the next release
- Otherwise: I've added my notes to release_notes.md
My changes do not need to be backported to a previous version
- Otherwise: Backport tracked by issue/PR #issue_or_pr
I have added all required tests (Unit tests, E2E tests)

Additional information

Additional PR information

trask

from joint review with Kaibo and Sreyas

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

src/main/java/com/microsoft/azure/functions/worker/broker/JavaFunctionBroker.java

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

trask

from joint review with @kaibocai @shreyas-gopalakrishna

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

src/main/java/com/microsoft/azure/functions/worker/broker/ParameterResolver.java

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

src/main/java/com/microsoft/azure/functions/worker/broker/JavaFunctionBroker.java

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

src/main/java/com/microsoft/azure/functions/worker/broker/JavaFunctionBroker.java

src/main/java/com/microsoft/azure/functions/worker/chain/InvocationChain.java

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

trask

we can discuss/review the Optional usage on the API pr

src/main/java/com/microsoft/azure/functions/worker/binding/ExecutionContextDataSource.java

* middleware support logics * refactor code * rename middleware chain * refactor code * fix error * refactor code * minor updates * remove used local varaible * APIs change accordingly * update method name * update method name

* decouple function definition and function executor * add function exceptions * remove unnecessary synchronized block * review updates * fix invocatioid error * rename factory get method * middleware support logics (#659) * middleware support logics * refactor code * rename middleware chain * refactor code * fix error * refactor code * minor updates * remove used local varaible * APIs change accordingly * update method name * update method name * update core lib version to 1.1.0

kaibocai requested review from anirudhgarg, TsuyoshiUshio, amamounelsayed and shreyas-gopalakrishna as code owners September 23, 2022 03:04

kaibocai requested a review from trask September 23, 2022 03:05

kaibocai force-pushed the kaibocai/middleware-pr2 branch from fa09cbd to 2d2e78a Compare September 23, 2022 03:56

middleware support logics

ca2296c

trask reviewed Sep 23, 2022

View reviewed changes

refactor code

207f3cc

kaibocai force-pushed the kaibocai/middleware-pr2 branch from 2d2e78a to 207f3cc Compare September 23, 2022 20:09

rename middleware chain

c4f67a7