optimize: spring boot compatible with file.conf and registry.conf (#6811)

[lyl2008dsg]

• I have registered the PR changes.

Ⅰ. Describe what this PR did

This PR implements a new class SeataFileConfigurationProvider in the seata-spring-autoconfigure-core module. This change ensures that both seata-spring-boot-starter and seata-spring-autoconfigure-server can read legacy configuration files (file.conf and registry.conf), addressing an issue where users upgrading from Spring to Spring Boot would only have their configuration loaded, causing confusion.

Ⅱ. Does this pull request fix one issue?

fixed #6811

This PR addresses the issue where seata-spring-boot-starter fails to load file.conf and registry.conf when users upgrade from Spring to Spring Boot.

Ⅲ. Why don't you add test cases (unit test/integration test)?

org.apache.seata.spring.boot.autoconfigure.SeataCoreAutoConfigurationTest

Ⅳ. Describe how to verify it

1. Use the seata-spring-boot-starter in a Spring Boot application.
2. Ensure that configurations from file.conf and registry.conf are correctly loaded alongside configurations.
3. Verify the behavior in both seata-spring-boot-starter and seata-spring-autoconfigure-server environments.

Ⅴ. Special notes for reviews

• This change is backward-compatible and allows users upgrading from Spring to Spring Boot to continue using their legacy configuration files without disruption.
• It would be helpful to review the relocation of SeataPropertiesLoader to the seata-spring-autoconfigure-core module, ensuring that legacy configuration files (file.conf and registry.conf) are still properly loaded in both seata-spring-boot-starter and seata-spring-autoconfigure-server environments. Additionally, please verify that this change maintains backward compatibility and ensures a smooth transition for users moving from Spring to Spring Boot.

[codecov]

Codecov Report

Attention: Patch coverage is 23.88060% with 51 lines in your changes missing coverage. Please review.

Project coverage is 52.64%. Comparing base (26950b6) to head (abd0de9).
Report is 1 commits behind head on 2.x.

Files with missing lines	Patch %	Lines
...n/java/org/apache/seata/common/store/LockMode.java	0.00%	16 Missing ⚠️
...ava/org/apache/seata/common/store/SessionMode.java	0.00%	16 Missing ⚠️
...ot/autoconfigure/loader/SeataPropertiesLoader.java	20.00%	12 Missing ⚠️
...he/seata/server/cluster/raft/RaftStateMachine.java	33.33%	2 Missing ⚠️
...e/seata/server/coordinator/DefaultCoordinator.java	33.33%	0 Missing and 2 partials ⚠️
...toconfigure/listener/SeataApplicationListener.java	90.90%	0 Missing and 1 partial ⚠️
...ache/seata/server/coordinator/RaftCoordinator.java	0.00%	1 Missing ⚠️
...apache/seata/server/lock/LockerManagerFactory.java	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6828      +/-   ##
============================================
- Coverage     52.66%   52.64%   -0.02%     
- Complexity     6643     6662      +19     
============================================
  Files          1126     1129       +3     
  Lines         40142    40159      +17     
  Branches       4707     4706       -1     
============================================
+ Hits          21139    21140       +1     
- Misses        16990    17001      +11     
- Partials       2013     2018       +5     

Files with missing lines	Coverage Δ
...e/seata/server/cluster/raft/RaftServerManager.java	86.13% <100.00%> (ø)
...org/apache/seata/server/session/SessionHelper.java	48.12% <ø> (ø)
...org/apache/seata/server/session/SessionHolder.java	55.12% <ø> (ø)
...ava/org/apache/seata/server/store/StoreConfig.java	65.78% <ø> (-9.54%)	⬇️
...toconfigure/listener/SeataApplicationListener.java	90.90% <90.90%> (ø)
...ache/seata/server/coordinator/RaftCoordinator.java	0.00% <0.00%> (ø)
...apache/seata/server/lock/LockerManagerFactory.java	84.21% <0.00%> (ø)
...he/seata/server/cluster/raft/RaftStateMachine.java	46.87% <33.33%> (+0.44%)	⬆️
...e/seata/server/coordinator/DefaultCoordinator.java	42.45% <33.33%> (ø)
...ot/autoconfigure/loader/SeataPropertiesLoader.java	62.85% <20.00%> (ø)
... and 2 more

... and 7 files with indirect coverage changes

[funky-eyes]

Why is this code being added?

[lyl2008dsg]

From an overall perspective, originalConfiguration.getConfig(rawDataId); is used to leverage getLatestConfig to fetch the configuration for rawDataId. After analysis, it indeed functionally covers getConfigFromSys. Therefore, it seems more appropriate to replace:

// 1. Get config value from the system property 
result = originalConfiguration.getConfigFromSys(rawDataId);

with:

// 1. Get config value from the system property 
result = originalConfiguration.getConfig(rawDataId);

instead of adding:

if (result == null) { 
  result = originalConfiguration.getConfig(rawDataId); 
}

[funky-eyes]

Then why still keep originalConfiguration.getConfigFromSys(rawDataId);?

[funky-eyes]

你目前的代码破坏了原本代码的目的，这段代码是为了先读环境变量和-D参数，然后再尝试读取spring中的配置，而你目前的改动变成了先读环境变量，在读配置中心，再读spring了，而这个SpringBootConfigurationProvider只是为了读取基本配置，配置中心中的配置有CURRENT_FILE_INSTANCE进行读取，不需要通过ORIGIN_FILE_INSTANCE_REGISTRY 也就是SpringBootConfigurationProvider读取
Your current code breaks the original purpose. This section of the code is intended to first read environment variables and -D parameters, and then attempt to read the configuration from Spring. However, with your modification, it now first reads from environment variables, then from the configuration center, and finally from Spring. The SpringBootConfigurationProvider is only meant to read basic configurations, and configurations in the configuration center are read via CURRENT_FILE_INSTANCE, so there’s no need to use ORIGIN_FILE_INSTANCE_REGISTRY, i.e., the SpringBootConfigurationProvider, to read them.

[lyl2008dsg]

Use SeataApplicationListener to set springConfigurableEnvironment,ensuring it's available during configuration retrieval.
This avoids the need for additional null checks and ensures proper loading timing.

[funky-eyes]

请合并最新的2.x分支
Please merge the latest 2.x branch

[funky-eyes]

LGTM