palashc commented on code in PR #1883:
URL: https://github.com/apache/phoenix/pull/1883#discussion_r1638592876


##########
phoenix-core-client/src/main/java/org/apache/phoenix/util/PhoenixRuntime.java:
##########
@@ -539,6 +541,18 @@ public static PTable getTable(Connection conn, @Nullable 
String tenantId, String
         return pconn.getTable(tenantId, fullTableName, timestamp);
     }
 
+    /**
+     * Returns the most recent PTable fetched from the server without updating 
the CQSI cache.
+     */
+    public static PTable getTableFromServerNoCache(Connection conn, byte[] 
schemaName,

Review Comment:
   @tkhurana region server endpoint (only called of this API currently) gets 
byte arrays for table name and schema name. CQSI.getTable also uses byte 
arrays. Is your suggestion to use String as argument types here - we will have 
to convert tableName and schemaName to String and then again to byte array 
eventually. 



-- 
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: [email protected]

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

Reply via email to