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

add dubbo plugin support #292

Merged
merged 6 commits into from
Nov 19, 2022
Merged

add dubbo plugin support #292

merged 6 commits into from
Nov 19, 2022

Conversation

oewang
Copy link
Contributor

@oewang oewang commented Nov 2, 2022

  • Support dubbo 2.5.0 to latest version

@oewang oewang closed this Nov 2, 2022
@oewang oewang reopened this Nov 2, 2022
@landyking
Copy link
Contributor

Good job! We will review it as soon as possible.

@landyking landyking self-requested a review November 2, 2022 03:32
@observeralone
Copy link
Contributor

don't use force push，the commit will merge when pr pass

@oewang
fix: add dubbo plugin
07f43fc 
- add return values collect switch
@oewang
Copy link
Contributor Author

oewang commented Nov 4, 2022

Okay,I get it.

@oewang
fix:
f43c68f 
- Resolve bug in asynchronous call trace and metrics
- Remove redundant configurations
- Additional unit tests
- Fix the problem that the local cache `configs` in the `AutoRefreshPluginConfigRegistry` class has no value
@oewang
fix:
3214226 
- Close the `scope` in the `after` method of the interceptor.
- Add a note about the json library used for parameters and return values.
oewang and others added 2 commits November 16, 2022 18:49
@oewang
fix:
943d1d8 
- `Scope` is not turned off in dubbo's asynchronous request.
- Error getting parameter type in `name` method of dubbo tool class.
@observeralone
Merge branch 'master' into dubbo-plugin
1aa6710
@observeralone observeralone merged commit 9ce54df into megaease:master Nov 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@observeralone observeralone observeralone approved these changes

@landyking landyking landyking approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oewang @landyking @observeralone