You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by zh...@apache.org on 2020/11/22 09:18:05 UTC

[shardingsphere] branch master updated: Update the sub class ShardingCTLShowBackendHandler of TextProtocolBackendHandler (#8279)

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

zhangliang 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 af2e03d  Update the sub class ShardingCTLShowBackendHandler of TextProtocolBackendHandler  (#8279)
af2e03d is described below

commit af2e03df8a952dea38e6313059c77ee541d16724
Author: sharpzhango <14...@qq.com>
AuthorDate: Sun Nov 22 17:17:50 2020 +0800

    Update the sub class ShardingCTLShowBackendHandler of TextProtocolBackendHandler  (#8279)
    
    * Update the sub class ShardingCTLShowBackendHandler of TextProtocolBackendHandler
    
    * update ShardingCTLShowBackendHandlerTest type from list to collection
---
 .../proxy/backend/text/sctl/show/ShardingCTLShowBackendHandler.java | 4 ++--
 .../backend/text/sctl/show/ShardingCTLShowBackendHandlerTest.java   | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/shardingsphere-proxy/shardingsphere-proxy-backend/src/main/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandler.java b/shardingsphere-proxy/shardingsphere-proxy-backend/src/main/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandler.java
index b16d8ea..235204c 100644
--- a/shardingsphere-proxy/shardingsphere-proxy-backend/src/main/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandler.java
+++ b/shardingsphere-proxy/shardingsphere-proxy-backend/src/main/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandler.java
@@ -31,7 +31,7 @@ import java.sql.SQLException;
 import java.sql.Types;
 import java.util.Arrays;
 import java.util.Collections;
-import java.util.List;
+import java.util.Collection;
 import java.util.Optional;
 
 /**
@@ -77,7 +77,7 @@ public final class ShardingCTLShowBackendHandler implements TextProtocolBackendH
     }
     
     @Override
-    public List<Object> getRowData() throws SQLException {
+    public Collection<Object> getRowData() throws SQLException {
         return Collections.singletonList(mergedResult.getValue(1, Object.class));
     }
 }
diff --git a/shardingsphere-proxy/shardingsphere-proxy-backend/src/test/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandlerTest.java b/shardingsphere-proxy/shardingsphere-proxy-backend/src/test/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandlerTest.java
index 2232e86..2c41316 100644
--- a/shardingsphere-proxy/shardingsphere-proxy-backend/src/test/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandlerTest.java
+++ b/shardingsphere-proxy/shardingsphere-proxy-backend/src/test/java/org/apache/shardingsphere/proxy/backend/text/sctl/show/ShardingCTLShowBackendHandlerTest.java
@@ -26,7 +26,7 @@ import org.apache.shardingsphere.transaction.core.TransactionType;
 import org.junit.Test;
 
 import java.sql.SQLException;
-import java.util.List;
+import java.util.Collection;
 
 import static org.hamcrest.CoreMatchers.instanceOf;
 import static org.hamcrest.CoreMatchers.is;
@@ -44,7 +44,7 @@ public final class ShardingCTLShowBackendHandlerTest {
         assertThat(actual, instanceOf(QueryResponse.class));
         assertThat(((QueryResponse) actual).getQueryHeaders().size(), is(1));
         backendHandler.next();
-        List<Object> rowData = backendHandler.getRowData();
+        Collection<Object> rowData = backendHandler.getRowData();
         assertThat(rowData.iterator().next(), is("LOCAL"));
     }
     
@@ -56,7 +56,7 @@ public final class ShardingCTLShowBackendHandlerTest {
         assertThat(actual, instanceOf(QueryResponse.class));
         assertThat(((QueryResponse) actual).getQueryHeaders().size(), is(1));
         backendHandler.next();
-        List<Object> rowData = backendHandler.getRowData();
+        Collection<Object> rowData = backendHandler.getRowData();
         assertThat(rowData.iterator().next(), is(0));
     }