shahrs87 commented on code in PR #1882:
URL: https://github.com/apache/phoenix/pull/1882#discussion_r1566502470


##########
phoenix-core/src/test/java/org/apache/phoenix/cache/ServerMetadataCacheIT.java:
##########
@@ -891,12 +891,14 @@ public void testSelectQueryAfterAlterIndex() throws 
Exception {
             stmt.executeQuery("SELECT k FROM " + tableName + " WHERE v1=1");
             Assert.assertEquals("Query on secondary key should have used 
index.", indexName, 
stmt.getQueryPlan().getTableRef().getTable().getTableName().toString());
 
-            //verify client-2 cache was updated with the index's base table 
metadata

Review Comment:
   > change needed because earlier I was calling MetadataClient.updateCache for 
the parent table but the correct way is to call it for the index.
   
   Where is this change being made?  @palashc 



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