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.

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

合并为增强引导类加载器加载的类而定制的Template、Transform #1272

Merged
merged 1 commit into from
Aug 12, 2023

Conversation

luanwenfei-venus
Copy link
Collaborator

【修复issue】#1234

【修改内容】1、新增AdviserScheduler,用于调度Sermant中的Adviser,AdviserInterface用于限制Adviser的能力 2、新增sermant-agentcore-god模块,用于放置需要全局可见的类ExecuteContext、Interceptor、AdviserScheduler、AdviserInterface 3、合并构造函数模板为TemplateForCtor、合并成员函数模板为TemplateForMember、合并静态函数模板为TemplateForStatic 4、合并Transformer为DefaultTransformer

【用例描述】不改变原有功能,不需新增用例

【自测情况】1、本地静态检查已过;2、测试用例暂未自动化

【影响范围】1、新增sermant-agentcore-god包,需要配合文档修改

@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Patch coverage: 0.59% and project coverage change: +1.44% 🎉

Comparison is base (b360faf) 45.56% compared to head (a98cf52) 47.00%.

Additional details and impacted files
@@             Coverage Diff              @@
##              1.0.x    #1272      +/-   ##
============================================
+ Coverage     45.56%   47.00%   +1.44%     
- Complexity     2924     2935      +11     
============================================
  Files           676      670       -6     
  Lines         14952    14621     -331     
  Branches       1679     1649      -30     
============================================
+ Hits           6813     6873      +60     
+ Misses         7296     6891     -405     
- Partials        843      857      +14     
Flag Coverage Δ
unittests 47.00% <0.59%> (+1.44%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
...om/huaweicloud/sermant/core/AgentCoreEntrance.java 0.00% <0.00%> (ø)
...ermant/core/plugin/agent/BufferedAgentBuilder.java 0.00% <0.00%> (ø)
...plugin/agent/collector/PluginCollectorManager.java 0.00% <0.00%> (ø)
.../core/plugin/agent/template/BaseAdviseHandler.java 0.00% <0.00%> (ø)
...ant/core/plugin/agent/template/DefaultAdviser.java 0.00% <0.00%> (ø)
...nt/core/plugin/agent/template/TemplateForCtor.java 0.00% <0.00%> (ø)
.../core/plugin/agent/template/TemplateForMember.java 0.00% <0.00%> (ø)
.../core/plugin/agent/template/TemplateForStatic.java 0.00% <0.00%> (ø)
...e/plugin/agent/transformer/DefaultTransformer.java 0.00% <0.00%> (ø)
...com/huaweicloud/sermant/core/utils/ClassUtils.java 51.61% <100.00%> (ø)

... and 19 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@luanwenfei-venus luanwenfei-venus self-assigned this Aug 10, 2023
@luanwenfei-venus luanwenfei-venus added area/framework Issues or PRs releated to sermant core service kind/refactor labels Aug 10, 2023
@luanwenfei-venus luanwenfei-venus merged commit 7fb8892 into sermant-io:1.0.x Aug 12, 2023
50 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/framework Issues or PRs releated to sermant core service
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants