Skip to content

Commit

Permalink
Remove spring dependencies from internal code (#3937)
Browse files Browse the repository at this point in the history
* add tech-support-qq-4.png

* Update README.md

* Enhance the user experience in the scenario of submitting duplicate keys

* Modify the key-value conflict exception prompt, adjust the code style

* fix:Remove spring dependencies from internal code

* doc(CHANGES.md): update CHANGES.md

Co-authored-by: Jason Song <nobodyiam@gmail.com>
  • Loading branch information
klboke and nobodyiam authored Sep 1, 2021
1 parent f909b8d commit ada6294
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ Apollo 1.10.0
* [switch apollo.config-service log from warning to info level](https://github.com/ctripcorp/apollo/pull/3884)
* [Make Access Key Timestamp check configurable](https://github.com/ctripcorp/apollo/pull/3908)
* [remove ctrip profile](https://github.com/ctripcorp/apollo/pull/3920)
* [Remove spring dependencies from internal code](https://github.com/apolloconfig/apollo/pull/3937)

------------------
All issues and pull requests are [here](https://github.com/ctripcorp/apollo/milestone/8?closed=1)
Original file line number Diff line number Diff line change
Expand Up @@ -39,8 +39,6 @@
import com.ctrip.framework.apollo.util.ExceptionUtil;
import com.google.common.collect.ImmutableMap;
import com.google.common.util.concurrent.RateLimiter;
import org.springframework.util.CollectionUtils;


/**
* @author Jason Song(song_s@ctrip.com)
Expand Down Expand Up @@ -186,10 +184,10 @@ public Set<String> getPropertyNames() {
// propertyNames include system property and system env might cause some compatibility issues, though that looks like the correct implementation.
Set<String> fromRepository = this.getPropertyNamesFromRepository();
Set<String> fromAdditional = this.getPropertyNamesFromAdditional();
if (CollectionUtils.isEmpty(fromRepository)) {
if (fromRepository == null || fromRepository.isEmpty()) {
return fromAdditional;
}
if (CollectionUtils.isEmpty(fromAdditional)) {
if (fromAdditional == null || fromAdditional.isEmpty()) {
return fromRepository;
}
Set<String> propertyNames = Sets
Expand Down

0 comments on commit ada6294

Please sign in to comment.