From 940bfa0f8880061d3d236150b2b592f14d94a4fd Mon Sep 17 00:00:00 2001 From: DerekYRC <44155264+DerekYRC@users.noreply.github.com> Date: Tue, 16 Aug 2022 10:32:12 +0800 Subject: [PATCH] Fix typo & Code optimization (#507) --- CHANGELOG.md | 1 + .../cloud/polaris/discovery/PolarisDiscoveryHandler.java | 2 +- .../cloud/polaris/discovery/PolarisServiceDiscovery.java | 4 ++-- .../config/PolarisRateLimitPropertiesAutoConfiguration.java | 4 ++-- .../cloud/polaris/ratelimit/constant/RateLimitConstant.java | 2 +- 5 files changed, 7 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 4e75f71b..c76ad2eb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,3 +8,4 @@ - [Bugfix: update byte-buddy scope test to compile](https://github.com/Tencent/spring-cloud-tencent/pull/495) - [feature:add @ConditionalOnConfigReflectEnabled annotation](https://github.com/Tencent/spring-cloud-tencent/pull/496) - [Feature: zuul supports polaris router](https://github.com/Tencent/spring-cloud-tencent/pull/502) +- [Fix typo & Code optimization](https://github.com/Tencent/spring-cloud-tencent/pull/507) diff --git a/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisDiscoveryHandler.java b/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisDiscoveryHandler.java index f96c7d3d..f3c8cbd7 100644 --- a/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisDiscoveryHandler.java +++ b/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisDiscoveryHandler.java @@ -92,7 +92,7 @@ public class PolarisDiscoveryHandler { * * @return service list */ - public ServicesResponse GetServices() { + public ServicesResponse getServices() { String namespace = polarisDiscoveryProperties.getNamespace(); GetServicesRequest request = new GetServicesRequest(); request.setNamespace(namespace); diff --git a/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisServiceDiscovery.java b/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisServiceDiscovery.java index 85c79897..6ef1eacf 100644 --- a/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisServiceDiscovery.java +++ b/spring-cloud-starter-tencent-polaris-discovery/src/main/java/com/tencent/cloud/polaris/discovery/PolarisServiceDiscovery.java @@ -67,10 +67,10 @@ public class PolarisServiceDiscovery { * @throws PolarisException polarisException */ public List getServices() throws PolarisException { - if (CollectionUtils.isEmpty(polarisDiscoveryHandler.GetServices().getServices())) { + if (CollectionUtils.isEmpty(polarisDiscoveryHandler.getServices().getServices())) { return Collections.emptyList(); } - return polarisDiscoveryHandler.GetServices().getServices().stream() + return polarisDiscoveryHandler.getServices().getServices().stream() .map(ServiceInfo::getService).collect(Collectors.toList()); } } diff --git a/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/config/PolarisRateLimitPropertiesAutoConfiguration.java b/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/config/PolarisRateLimitPropertiesAutoConfiguration.java index e00d6f63..672a1dae 100644 --- a/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/config/PolarisRateLimitPropertiesAutoConfiguration.java +++ b/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/config/PolarisRateLimitPropertiesAutoConfiguration.java @@ -20,9 +20,9 @@ package com.tencent.cloud.polaris.ratelimit.config; import com.tencent.cloud.polaris.context.ConditionalOnPolarisEnabled; import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean; +import org.springframework.boot.context.properties.EnableConfigurationProperties; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Configuration; -import org.springframework.context.annotation.Import; /** * Properties auto configuration of rate limit. @@ -31,7 +31,7 @@ import org.springframework.context.annotation.Import; */ @Configuration(proxyBeanMethods = false) @ConditionalOnPolarisEnabled -@Import({PolarisRateLimitProperties.class}) +@EnableConfigurationProperties(PolarisRateLimitProperties.class) public class PolarisRateLimitPropertiesAutoConfiguration { @Bean diff --git a/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/constant/RateLimitConstant.java b/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/constant/RateLimitConstant.java index 22f11ed0..814bed56 100644 --- a/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/constant/RateLimitConstant.java +++ b/spring-cloud-starter-tencent-polaris-ratelimit/src/main/java/com/tencent/cloud/polaris/ratelimit/constant/RateLimitConstant.java @@ -35,7 +35,7 @@ public final class RateLimitConstant { /** * Info of rate limit. */ - public static String QUOTA_LIMITED_INFO = "The request is deny by rate limit because the throttling threshold is reached"; + public static String QUOTA_LIMITED_INFO = "The request is denied by rate limit because the throttling threshold is reached"; /** * The build in label method.