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

chengzhang 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 9c4fa901b9b Use CaseInsensitiveMap to replace toLowerCase method call 
(#32595)
9c4fa901b9b is described below

commit 9c4fa901b9b5cd1f1b8fcef4f31167801e408778
Author: Zhengqiang Duan <[email protected]>
AuthorDate: Tue Aug 20 08:12:55 2024 +0800

    Use CaseInsensitiveMap to replace toLowerCase method call (#32595)
---
 .../database/schema/model/ShardingSphereTable.java | 33 +++++++++++-----------
 1 file changed, 16 insertions(+), 17 deletions(-)

diff --git 
a/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/model/ShardingSphereTable.java
 
b/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/model/ShardingSphereTable.java
index a7e37fdac86..bd5eb1db5a5 100644
--- 
a/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/model/ShardingSphereTable.java
+++ 
b/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/model/ShardingSphereTable.java
@@ -17,6 +17,7 @@
 
 package org.apache.shardingsphere.infra.metadata.database.schema.model;
 
+import com.cedarsoftware.util.CaseInsensitiveMap;
 import lombok.EqualsAndHashCode;
 import lombok.Getter;
 import lombok.ToString;
@@ -25,7 +26,6 @@ import 
org.apache.shardingsphere.infra.database.core.metadata.database.enums.Tab
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
-import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 
@@ -63,7 +63,7 @@ public final class ShardingSphereTable {
         this.columns = createColumns(columns);
         this.indexes = createIndexes(indexes);
         this.constraints = createConstraints(constraints);
-        this.type = TableType.TABLE;
+        type = TableType.TABLE;
     }
     
     public ShardingSphereTable(final String name, final 
Collection<ShardingSphereColumn> columns,
@@ -76,13 +76,12 @@ public final class ShardingSphereTable {
     }
     
     private Map<String, ShardingSphereColumn> createColumns(final 
Collection<ShardingSphereColumn> columns) {
-        Map<String, ShardingSphereColumn> result = new 
LinkedHashMap<>(columns.size(), 1F);
+        Map<String, ShardingSphereColumn> result = new 
CaseInsensitiveMap<>(columns.size(), 1F);
         for (ShardingSphereColumn each : columns) {
-            String lowerColumnName = each.getName().toLowerCase();
-            result.put(lowerColumnName, each);
+            result.put(each.getName(), each);
             columnNames.add(each.getName());
             if (each.isPrimaryKey()) {
-                primaryKeyColumns.add(lowerColumnName);
+                primaryKeyColumns.add(each.getName());
             }
             if (each.isVisible()) {
                 visibleColumns.add(each.getName());
@@ -92,17 +91,17 @@ public final class ShardingSphereTable {
     }
     
     private Map<String, ShardingSphereIndex> createIndexes(final 
Collection<ShardingSphereIndex> indexes) {
-        Map<String, ShardingSphereIndex> result = new 
LinkedHashMap<>(indexes.size(), 1F);
+        Map<String, ShardingSphereIndex> result = new 
CaseInsensitiveMap<>(indexes.size(), 1F);
         for (ShardingSphereIndex each : indexes) {
-            result.put(each.getName().toLowerCase(), each);
+            result.put(each.getName(), each);
         }
         return result;
     }
     
     private Map<String, ShardingSphereConstraint> createConstraints(final 
Collection<ShardingSphereConstraint> constraints) {
-        Map<String, ShardingSphereConstraint> result = new 
LinkedHashMap<>(constraints.size(), 1F);
+        Map<String, ShardingSphereConstraint> result = new 
CaseInsensitiveMap<>(constraints.size(), 1F);
         for (ShardingSphereConstraint each : constraints) {
-            result.put(each.getName().toLowerCase(), each);
+            result.put(each.getName(), each);
         }
         return result;
     }
@@ -113,7 +112,7 @@ public final class ShardingSphereTable {
      * @param column column meta data
      */
     public void putColumn(final ShardingSphereColumn column) {
-        columns.put(column.getName().toLowerCase(), column);
+        columns.put(column.getName(), column);
     }
     
     /**
@@ -123,7 +122,7 @@ public final class ShardingSphereTable {
      * @return column meta data
      */
     public ShardingSphereColumn getColumn(final String columnName) {
-        return columns.get(columnName.toLowerCase());
+        return columns.get(columnName);
     }
     
     /**
@@ -142,7 +141,7 @@ public final class ShardingSphereTable {
      * @return whether contains column or not
      */
     public boolean containsColumn(final String columnName) {
-        return null != columnName && 
columns.containsKey(columnName.toLowerCase());
+        return null != columnName && columns.containsKey(columnName);
     }
     
     /**
@@ -151,7 +150,7 @@ public final class ShardingSphereTable {
      * @param index index meta data
      */
     public void putIndex(final ShardingSphereIndex index) {
-        indexes.put(index.getName().toLowerCase(), index);
+        indexes.put(index.getName(), index);
     }
     
     /**
@@ -160,7 +159,7 @@ public final class ShardingSphereTable {
      * @param indexName index name
      */
     public void removeIndex(final String indexName) {
-        indexes.remove(indexName.toLowerCase());
+        indexes.remove(indexName);
     }
     
     /**
@@ -170,7 +169,7 @@ public final class ShardingSphereTable {
      * @return index meta data
      */
     public ShardingSphereIndex getIndex(final String indexName) {
-        return indexes.get(indexName.toLowerCase());
+        return indexes.get(indexName);
     }
     
     /**
@@ -189,7 +188,7 @@ public final class ShardingSphereTable {
      * @return whether contains index or not
      */
     public boolean containsIndex(final String indexName) {
-        return null != indexName && 
indexes.containsKey(indexName.toLowerCase());
+        return null != indexName && indexes.containsKey(indexName);
     }
     
     /**

Reply via email to