Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

Add biz system properties support 394 feature #452

Conversation

qq290584697
Copy link
Collaborator

完成issue #394
支持业务模块和基座之间的系统变量隔离
其中基座的系统变量为业务模块变量的默认值
即:

  1. 业务模块存在该系统变量,直接返回
  2. 业务模块不存在该系统变量,返回基座的系统变量的值

考虑到并非所有场景都需要将系统变量隔离
如需使用,需要在基座应用的main方法中,插入下列代码

MultiBizProperties.initSystem()

@sofastack-cla sofastack-cla bot added size/XL and removed size/L labels Jan 17, 2024
Copy link

codecov bot commented Jan 17, 2024

Codecov Report

Attention: 35 lines in your changes are missing coverage. Please review.

Comparison is base (b83af8a) 80.93% compared to head (1e59b6f) 81.00%.
Report is 9 commits behind head on master.

Files Patch % Lines
...ofa/serverless/common/util/MultiBizProperties.java 81.38% 25 Missing and 10 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #452      +/-   ##
============================================
+ Coverage     80.93%   81.00%   +0.06%     
  Complexity       64       64              
============================================
  Files            45       46       +1     
  Lines          1028     1216     +188     
  Branches         90      113      +23     
============================================
+ Hits            832      985     +153     
- Misses          145      170      +25     
- Partials         51       61      +10     

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

Copy link
Collaborator

@lvjing2 lvjing2 left a comment

Choose a reason for hiding this comment

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

LGTM

@lvjing2 lvjing2 merged commit 2ad5a24 into sofastack:master Jan 18, 2024
4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants