jpisaac commented on code in PR #1748:
URL: https://github.com/apache/phoenix/pull/1748#discussion_r1419676088


##########
phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java:
##########
@@ -604,6 +608,8 @@ public static PName newPName(byte[] keyBuffer, int 
keyOffset, int keyLength) {
 
     private MetricsMetadataSource metricsSource;
     private long metadataCacheInvalidationTimeoutMs;
+    private Connection invalidateMetadataCacheConnection = null;

Review Comment:
   @shahrs87 As discussed offline, wondering whether this connection for 
invalidation can be wrapped inside CQSI. It might be useful to provide an 
interface through CQSI. This will be in line with our current packaging of 
hConnections inside CQSI. Another advantage will be we can then use 
PhoenixConnection/CQSI to invoke this from out-of-band tools and clients too.
   To begin with, the invalidation API can be exposed only for server-side 
connection and be a no-op for other clients/profiles. 



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