gokceni commented on a change in pull request #1135:
URL: https://github.com/apache/phoenix/pull/1135#discussion_r572485854



##########
File path: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/AlterAddCascadeIndexIT.java
##########
@@ -172,6 +172,41 @@ public void testAlterDBOAddCascadeIndexAllUpsert() throws 
Exception {
 
     }
 
+    // Test with ALTER TABLE/VIEW CASCADE INDEX ALL with no indexes
+    @Test
+    public void testAlterDBOAddCascadeIndexAll_noIndexes() throws Exception {
+        String schemaName = "S_"+generateUniqueName();
+        String tableName = "T_"+generateUniqueName();
+        String viewName = "V_"+generateUniqueName();
+
+        String fullViewName = SchemaUtil.getQualifiedTableName(schemaName, 
viewName);
+        String fullTableName = SchemaUtil.getQualifiedTableName(schemaName, 
tableName);
+
+        conn.createStatement().execute("CREATE TABLE IF NOT EXISTS " + 
fullTableName + " (\n" +
+                "      state CHAR(2) NOT NULL,\n" +
+                "      city VARCHAR NOT NULL,\n" +
+                "      population BIGINT,\n" +
+                "      CONSTRAINT my_pk PRIMARY KEY (state, city)) " + 
tableDDLOptions);
+
+        if(isViewIndex) {

Review comment:
       Is it possible to rename this as isViewScenario?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to