This is an automated email from the ASF dual-hosted git repository.

zhaojinchao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 75968f636b2 Remove @beta usage (#23983)
75968f636b2 is described below

commit 75968f636b2418156cf63641b07515203c659bf9
Author: Liang Zhang <[email protected]>
AuthorDate: Fri Feb 3 23:19:17 2023 +0800

    Remove @beta usage (#23983)
---
 .../sharding/range/BoundaryBasedRangeShardingAlgorithm.java | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git 
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/algorithm/sharding/range/BoundaryBasedRangeShardingAlgorithm.java
 
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/algorithm/sharding/range/BoundaryBasedRangeShardingAlgorithm.java
index 3d3409c6e26..ab3d287a66c 100644
--- 
a/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/algorithm/sharding/range/BoundaryBasedRangeShardingAlgorithm.java
+++ 
b/features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/algorithm/sharding/range/BoundaryBasedRangeShardingAlgorithm.java
@@ -19,7 +19,6 @@ package 
org.apache.shardingsphere.sharding.algorithm.sharding.range;
 
 import com.google.common.base.Splitter;
 import com.google.common.collect.Range;
-import com.google.common.primitives.Longs;
 import 
org.apache.shardingsphere.infra.util.exception.ShardingSpherePreconditions;
 import 
org.apache.shardingsphere.sharding.exception.algorithm.sharding.ShardingAlgorithmInitializationException;
 
@@ -40,8 +39,8 @@ public final class BoundaryBasedRangeShardingAlgorithm 
extends AbstractRangeShar
     @Override
     public Map<Integer, Range<Comparable<?>>> calculatePartitionRange(final 
Properties props) {
         
ShardingSpherePreconditions.checkState(props.containsKey(SHARDING_RANGES_KEY), 
() -> new ShardingAlgorithmInitializationException(getType(), "Sharding ranges 
cannot be null."));
-        List<Long> partitionRanges = 
Splitter.on(",").trimResults().splitToList(props.getProperty(SHARDING_RANGES_KEY))
-                
.stream().map(Longs::tryParse).filter(Objects::nonNull).sorted().collect(Collectors.toList());
+        List<Long> partitionRanges = 
Splitter.on(",").trimResults().splitToList(props.getProperty(SHARDING_RANGES_KEY)).stream()
+                
.map(this::parseLong).filter(Objects::nonNull).sorted().collect(Collectors.toList());
         ShardingSpherePreconditions.checkState(!partitionRanges.isEmpty(), () 
-> new ShardingAlgorithmInitializationException(getType(), "Sharding ranges can 
not be empty."));
         Map<Integer, Range<Comparable<?>>> result = new 
HashMap<>(partitionRanges.size() + 1, 1);
         for (int i = 0; i < partitionRanges.size(); i++) {
@@ -59,6 +58,14 @@ public final class BoundaryBasedRangeShardingAlgorithm 
extends AbstractRangeShar
         return result;
     }
     
+    private Long parseLong(final String value) {
+        try {
+            return Long.parseLong(value);
+        } catch (final NumberFormatException ex) {
+            return null;
+        }
+    }
+    
     @Override
     public String getType() {
         return "BOUNDARY_RANGE";

Reply via email to