tkhurana commented on code in PR #2026:
URL: https://github.com/apache/phoenix/pull/2026#discussion_r1862739499


##########
phoenix-core-client/src/main/java/org/apache/phoenix/util/ViewUtil.java:
##########
@@ -886,4 +893,44 @@ public static void 
addTagsToPutsForViewAlteredProperties(List<Mutation> tableMet
 
         }
     }
+
+    /**
+     * Retrieves the list of unique view index ids from SYSTEM.CATALOG for a 
given view index table
+     * @param connection - Phoenix Connection
+     * @param tableName - name of the view index table for which view index 
ids need to be retrieved

Review Comment:
   It looks like this is the physical name of the view Index table and not the 
logical name. Can you update the API document to clarify this.



-- 
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: issues-unsubscr...@phoenix.apache.org

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

Reply via email to