You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by wu...@apache.org on 2021/06/10 15:24:50 UTC
[shardingsphere] branch master updated: Remove useless codes
(#10756)
This is an automated email from the ASF dual-hosted git repository.
wuweijie 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 c20eddf Remove useless codes (#10756)
c20eddf is described below
commit c20eddf83c5e7f12d33e3277745375cc579d1330
Author: Liang Zhang <te...@163.com>
AuthorDate: Thu Jun 10 23:24:13 2021 +0800
Remove useless codes (#10756)
---
.../apache/shardingsphere/sharding/rule/ShardingRule.java | 14 --------------
.../shardingsphere/sharding/rule/ShardingRuleTest.java | 10 ----------
2 files changed, 24 deletions(-)
diff --git a/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java b/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
index e462de8..0c6facb 100644
--- a/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
+++ b/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/main/java/org/apache/shardingsphere/sharding/rule/ShardingRule.java
@@ -430,20 +430,6 @@ public final class ShardingRule implements FeatureRule, SchemaRule, DataNodeCont
}
/**
- * Find data node by data source and logic table.
- *
- * @param dataSourceName data source name
- * @param logicTableName logic table name
- * @return data node
- */
- public DataNode getDataNode(final String dataSourceName, final String logicTableName) {
- TableRule tableRule = getTableRule(logicTableName);
- return tableRule.getActualDataNodes().stream().filter(each -> dataSourceNames.contains(each.getDataSourceName())
- && each.getDataSourceName().equals(dataSourceName)).findFirst()
- .orElseThrow(() -> new ShardingSphereConfigurationException("Cannot find actual data node for data source name: '%s' and logic table name: '%s'", dataSourceName, logicTableName));
- }
-
- /**
* Get sharding logic table names.
*
* @param logicTableNames logic table names
diff --git a/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/test/java/org/apache/shardingsphere/sharding/rule/ShardingRuleTest.java b/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/test/java/org/apache/shardingsphere/sharding/rule/ShardingRuleTest.java
index cf0473d..4a704c6 100644
--- a/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/test/java/org/apache/shardingsphere/sharding/rule/ShardingRuleTest.java
+++ b/shardingsphere-features/shardingsphere-sharding/shardingsphere-sharding-core/src/test/java/org/apache/shardingsphere/sharding/rule/ShardingRuleTest.java
@@ -270,16 +270,6 @@ public final class ShardingRuleTest {
}
@Test
- public void assertGetDataNodeByDataSourceAndLogicTable() {
- assertThat(createMaximumShardingRule().getDataNode("ds_1", "logic_table"), is(new DataNode("ds_1.table_0")));
- }
-
- @Test(expected = ShardingSphereConfigurationException.class)
- public void assertGetDataNodeByLogicTableFailureWithDataSourceName() {
- createMaximumShardingRule().getDataNode("ds_3", "logic_table");
- }
-
- @Test
public void assertGetShardingLogicTableNames() {
ShardingRule actual = createMaximumShardingRule();
assertThat(actual.getShardingLogicTableNames(Arrays.asList("LOGIC_TABLE", "BROADCAST_TABLE")), is(Collections.singletonList("LOGIC_TABLE")));