Skip to content

feat: adapt to polaris-java ReportClientRequestCustomizer plugin for config watch reporting - #1813

Merged
fuyuwei01 merged 12 commits into
Tencent:2024from
fuyuwei01:dev/config-listener-report
Aug 13, 2026
Merged

feat: adapt to polaris-java ReportClientRequestCustomizer plugin for config watch reporting#1813
fuyuwei01 merged 12 commits into
Tencent:2024from
fuyuwei01:dev/config-listener-report

Conversation

@fuyuwei01

Copy link
Copy Markdown
Collaborator

PR Type

Describe what this PR does for and how you did.

Adding the issue link (#xxx) if possible.

Note

Checklist

  • Add information of this PR to CHANGELOG.md in root of project.
  • Add documentation in javadoc or comment below the PR if necessary.

fishtailfu added 7 commits August 11, 2026 16:51
…eporting

- Upgrade polaris.version from 2.1.2.0-SNAPSHOT to 2.1.2.0
- Enable clientReporter in ConfigurationModifier.modify()
- Add ClientReporterConfigImpl mock to ConfigurationModifierTest
- Add clientReporter verification to test assertions
@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.12%. Comparing base (859b025) to head (83251ec).

Additional details and impacted files
@@             Coverage Diff              @@
##               2024    #1813      +/-   ##
============================================
+ Coverage     55.07%   55.12%   +0.04%     
  Complexity      152      152              
============================================
  Files           482      482              
  Lines         14110    14125      +15     
  Branches       1766     1766              
============================================
+ Hits           7771     7786      +15     
  Misses         5711     5711              
  Partials        628      628              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fuyuwei01 fuyuwei01 changed the title feat: adapt to polaris-java CLIENT_REPORTER plugin for config watch reporting feat: adapt to polaris-java ReportClientRequestCustomizer plugin for config watch reporting Aug 12, 2026
Comment thread spring-cloud-tencent-dependencies/pom.xml Outdated
Comment thread pom.xml Outdated
@fuyuwei01
fuyuwei01 force-pushed the dev/config-listener-report branch from e7bfb7e to cb6c43c Compare August 12, 2026 07:53
@fuyuwei01
fuyuwei01 force-pushed the dev/config-listener-report branch from be3ecb4 to 2dce251 Compare August 12, 2026 08:14
@SkyeBeFreeman SkyeBeFreeman added this to the 2.2.0.x milestone Aug 12, 2026
SkyeBeFreeman
SkyeBeFreeman previously approved these changes Aug 12, 2026

@SkyeBeFreeman SkyeBeFreeman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@SkyeBeFreeman SkyeBeFreeman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@fuyuwei01
fuyuwei01 merged commit 1c6f999 into Tencent:2024 Aug 13, 2026
9 checks passed
peerless1024 pushed a commit that referenced this pull request Aug 26, 2026
…config watch reporting (#1813)

* feat: adapt to polaris-java CLIENT_REPORTER plugin for config watch reporting

- Upgrade polaris.version from 2.1.2.0-SNAPSHOT to 2.1.2.0
- Enable clientReporter in ConfigurationModifier.modify()
- Add ClientReporterConfigImpl mock to ConfigurationModifierTest
- Add clientReporter verification to test assertions

* fix: enable config watch client reporter plugin in SCT

* fix: add config report property

* fix: add config report property

* add CHANGELOG.md

* fix pom

* refactor: rename config report request customizer

* fix CHANGELOG.md

* fix REVISION

* fix properties

* refactor: use generic request customizer config

* fix: align request customizer config type

---------

Co-authored-by: fishtailfu <fishtailfu@tencent.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants