palashc commented on code in PR #1882:
URL: https://github.com/apache/phoenix/pull/1882#discussion_r1566412712
##########
phoenix-core-server/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java:
##########
@@ -3904,6 +3904,13 @@ private MetaDataMutationResult
dropIndexes(RegionCoprocessorEnvironment env, PTa
// that a change in index state doesn't
// occur while we're dropping it.
acquireLock(region, indexKey, locks);
+ // invalidate server metadata cache when dropping index
+ List<InvalidateServerMetadataCacheRequest> requests = new
ArrayList<>();
+ requests.add(new InvalidateServerMetadataCacheRequest(tenantId,
Review Comment:
@shahrs87 Yes, we need to invalidate the entry for any indexes which we are
dropping here. Are you suggesting I change the comment here?
--
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]
