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

[#9653] Add plugin package class requirement filter #9658

Merged

Conversation

donghun-cho
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jan 26, 2023

Codecov Report

Merging #9658 (65a2a29) into master (ea23f30) will increase coverage by 0.00%.
The diff coverage is 70.17%.

@@            Coverage Diff            @@
##             master    #9658   +/-   ##
=========================================
  Coverage     38.93%   38.94%           
- Complexity    11909    11917    +8     
=========================================
  Files          3493     3494    +1     
  Lines         93715    93761   +46     
  Branches      10509    10515    +6     
=========================================
+ Hits          36487    36514   +27     
- Misses        54057    54071   +14     
- Partials       3171     3176    +5     
Impacted Files Coverage Δ
...ler/plugin/DefaultProfilerPluginContextLoader.java 26.43% <0.00%> (-1.28%) ⬇️
.../navercorp/pinpoint/profiler/plugin/PluginJar.java 48.27% <0.00%> (-1.73%) ⬇️
...strument/classloading/PlainClassLoaderHandler.java 53.45% <66.66%> (-0.05%) ⬇️
.../navercorp/pinpoint/profiler/plugin/JarPlugin.java 47.05% <66.66%> (+0.39%) ⬆️
...er/plugin/PluginPackageClassRequirementFilter.java 80.64% <80.64%> (ø)
...rcorp/pinpoint/profiler/plugin/PluginManifest.java 70.83% <85.71%> (+0.83%) ⬆️
...vercorp/pinpoint/profiler/plugin/PluginConfig.java 70.58% <100.00%> (+3.92%) ⬆️
...vercorp/pinpoint/rpc/packet/ClientClosePacket.java 46.66% <0.00%> (-6.67%) ⬇️
...or/metric/response/ReuseResponseTimeCollector.java 88.63% <0.00%> (-4.55%) ⬇️
...tor/metric/datasource/DefaultDataSourceMetric.java 85.18% <0.00%> (-3.71%) ⬇️
... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@donghun-cho donghun-cho force-pushed the plugin-packageRequirementFilter branch from 882c3be to bac757a Compare January 31, 2023 02:46
@donghun-cho donghun-cho linked an issue Jan 31, 2023 that may be closed by this pull request
@donghun-cho donghun-cho force-pushed the plugin-packageRequirementFilter branch from bac757a to 65a2a29 Compare January 31, 2023 06:30
@donghun-cho donghun-cho self-assigned this Jan 31, 2023
@donghun-cho donghun-cho merged commit 0e54106 into pinpoint-apm:master Jan 31, 2023
@donghun-cho donghun-cho deleted the plugin-packageRequirementFilter branch August 31, 2023 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add PluginPackageClassRequirementFilter
3 participants