-
Notifications
You must be signed in to change notification settings - Fork 26.4k
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
[System properties] Add the system properties tool, unified system configuration items #13307
Conversation
All configuration items that use System.getProperty can be divided into three categories
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 3.3 #13307 +/- ##
==========================================
+ Coverage 40.68% 40.71% +0.03%
==========================================
Files 1806 1807 +1
Lines 75805 75834 +29
Branches 10636 10641 +5
==========================================
+ Hits 30841 30878 +37
+ Misses 40639 40637 -2
+ Partials 4325 4319 -6 ☔ View full report in Codecov by Sentry. |
dubbo-common/src/main/java/org/apache/dubbo/common/constants/CommonConstants.java
Show resolved
Hide resolved
dubbo-plugin/dubbo-native/src/main/java/org/apache/dubbo/aot/generate/AotProcessor.java
Outdated
Show resolved
Hide resolved
...ubbo-native/src/main/java/org/apache/dubbo/aot/generate/ReflectConfigMetadataRepository.java
Outdated
Show resolved
Hide resolved
@CrazyHZM The native image CI pass, another CI does not pass, and it does not pass in any other PR. Please review again~ |
Kudos, SonarCloud Quality Gate passed! |
@CrazyHZM All CI have passed. Please review again~ |
What is the purpose of the change
Add the system properties tool, unified system configuration items.
Brief changelog
Add class of SystemPropertyConfigUtils and SystemPropertyConfigUtilsTest.
Verifying this change
Checklist