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

[type:refactor] refactor log module #4526

Merged
merged 4 commits into from
Apr 3, 2023

Conversation

moremind
Copy link
Member

@moremind moremind commented Apr 1, 2023

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@moremind moremind changed the title [type:refactor] rename log module [type:refactor] refactor log module Apr 1, 2023
@codecov-commenter
Copy link

codecov-commenter commented Apr 3, 2023

Codecov Report

Merging #4526 (a07fd1c) into master (9b726ee) will increase coverage by 0.04%.
The diff coverage is 40.90%.

@@             Coverage Diff              @@
##             master    #4526      +/-   ##
============================================
+ Coverage     68.39%   68.44%   +0.04%     
- Complexity     7620     7622       +2     
============================================
  Files          1028     1028              
  Lines         29440    29440              
  Branches       2666     2666              
============================================
+ Hits          20136    20150      +14     
+ Misses         7710     7694      -16     
- Partials       1594     1596       +2     
Impacted Files Coverage Δ
...in/aliyun/sls/collector/AliyunSlsLogCollector.java 80.00% <ø> (ø)
...g/clickhouse/collector/ClickHouseLogCollector.java 80.00% <ø> (ø)
...u/plugin/logging/common/AbstractLoggingPlugin.java 0.00% <0.00%> (ø)
...logging/common/collector/AbstractLogCollector.java 0.00% <0.00%> (ø)
...logging/common/entity/CommonLoggingRuleHandle.java 0.00% <0.00%> (ø)
.../logging/desensitize/api/matcher/KeyWordMatch.java 100.00% <ø> (ø)
...ticsearch/collector/ElasticSearchLogCollector.java 80.00% <ø> (ø)
...gin/logging/kafka/collector/KafkaLogCollector.java 80.00% <ø> (ø)
...n/logging/pulsar/collector/PulsarLogCollector.java 80.00% <ø> (ø)
...gging/rocketmq/collector/RocketMQLogCollector.java 80.00% <ø> (ø)
... and 9 more

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yu199195 yu199195 added this to the 2.6.0 milestone Apr 3, 2023
@yu199195 yu199195 merged commit 114f1db into apache:master Apr 3, 2023
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.

3 participants