You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by du...@apache.org on 2023/03/13 10:43:36 UTC

[shardingsphere] branch master updated: Fix revise engine data source NPE. (#24581)

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

duanzhengqiang 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 f984531229f Fix revise engine data source NPE. (#24581)
f984531229f is described below

commit f984531229f238ae4bb85d7d784c5b83e5e6c806
Author: Chuxin Chen <ch...@qq.com>
AuthorDate: Mon Mar 13 18:43:23 2023 +0800

    Fix revise engine data source NPE. (#24581)
---
 .../metadata/database/schema/reviser/MetaDataReviseEngine.java     | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/MetaDataReviseEngine.java b/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/MetaDataReviseEngine.java
index 91b3a048fcd..2ac54d9778c 100644
--- a/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/MetaDataReviseEngine.java
+++ b/infra/common/src/main/java/org/apache/shardingsphere/infra/metadata/database/schema/reviser/MetaDataReviseEngine.java
@@ -20,8 +20,8 @@ package org.apache.shardingsphere.infra.metadata.database.schema.reviser;
 import lombok.RequiredArgsConstructor;
 import org.apache.shardingsphere.infra.database.type.DatabaseType;
 import org.apache.shardingsphere.infra.metadata.database.schema.builder.GenericSchemaBuilderMaterial;
-import org.apache.shardingsphere.infra.metadata.database.schema.reviser.schema.SchemaMetaDataReviseEngine;
 import org.apache.shardingsphere.infra.metadata.database.schema.loader.model.SchemaMetaData;
+import org.apache.shardingsphere.infra.metadata.database.schema.reviser.schema.SchemaMetaDataReviseEngine;
 import org.apache.shardingsphere.infra.rule.ShardingSphereRule;
 
 import javax.sql.DataSource;
@@ -48,8 +48,9 @@ public final class MetaDataReviseEngine {
     public Map<String, SchemaMetaData> revise(final Map<String, SchemaMetaData> schemaMetaDataMap, final GenericSchemaBuilderMaterial material) {
         Map<String, SchemaMetaData> result = new LinkedHashMap<>(schemaMetaDataMap.size(), 1);
         for (Entry<String, SchemaMetaData> entry : schemaMetaDataMap.entrySet()) {
-            DatabaseType databaseType = material.getStorageTypes().get(entry.getKey());
-            DataSource dataSource = material.getDataSourceMap().get(entry.getKey());
+            // TODO establish a corresponding relationship between tables and data sources
+            DatabaseType databaseType = material.getStorageTypes().values().stream().findFirst().orElse(null);
+            DataSource dataSource = material.getDataSourceMap().values().stream().findFirst().orElse(null);
             result.put(entry.getKey(), new SchemaMetaDataReviseEngine(rules, material.getProps(), databaseType, dataSource).revise(entry.getValue()));
         }
         return result;