Maxwell-Guo commented on code in PR #3718:
URL: https://github.com/apache/cassandra/pull/3718#discussion_r1875866620


##########
test/unit/org/apache/cassandra/cql3/CQLTester.java:
##########
@@ -1543,6 +1578,16 @@ protected static ResultMessage schemaChange(String query)
         }
     }
 
+    protected TableMetadata getTableMetadata(String table)
+    {
+        return Schema.instance.getTableMetadata(KEYSPACE, table);
+    }
+
+    protected TableMetadata getTableMetadata(String keyspace, String table)
+    {
+        return Schema.instance.getTableMetadata(keyspace, table);
+    }
+

Review Comment:
   @blerer 
   I removed 
   ``` protected TableMetadata getTableMetadata(String table)
       {
           return Schema.instance.getTableMetadata(KEYSPACE, table);
       }
   ```
   But kept 
   ```
   protected TableMetadata getTableMetadata(String keyspace, String table)
       {
           return Schema.instance.getTableMetadata(keyspace, table);
       }
   ```
   as ``  getTableMetadata(String keyspace, String table)`` is used by some 
test cases, is it ok ?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to