Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Agent: remove MethodInvocationResult #23067

Closed
jiangML opened this issue Dec 23, 2022 · 0 comments
Closed

Agent: remove MethodInvocationResult #23067

jiangML opened this issue Dec 23, 2022 · 0 comments
Assignees
@jiangML
Labels
feature: agent shardingsphere agent type: refactor
Milestone
5.3.1

Comments

@jiangML
Copy link
Contributor

jiangML commented Dec 23, 2022 •
edited
Loading

Feature Request

For English only, other languages will not accept.

Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot make decision by current information, we will close it.

Please answer these questions before submitting your issue. Thanks!

Is your feature request related to a problem?

Agent does not need to change the original execution result.

Describe the feature you would like.

1 Remove MethodInvocationResult.
2 The method afterMethod() of InstanceMethodAdvice and StaticMethodAdvice adds the original execution result as an input parameter.
@jiangML jiangML mentioned this issue Dec 23, 2022
Merged
6 tasks
@zhaojinchao95 zhaojinchao95 added this to the 5.3.1 milestone Dec 24, 2022
@sandynz sandynz closed this as completed Dec 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jiangML jiangML

Labels
feature: agent shardingsphere agent type: refactor
Projects
None yet
Milestone
5.3.1
Development

No branches or pull requests
3 participants
@jiangML @zhaojinchao95 @sandynz