tolbertam commented on code in PR #3917:
URL: https://github.com/apache/cassandra/pull/3917#discussion_r2083578597


##########
src/java/org/apache/cassandra/cql3/QueryProcessor.java:
##########
@@ -131,6 +128,16 @@ public class QueryProcessor implements QueryHandler
                     DatabaseDescriptor.getPreparedStatementsCacheSizeMiB());
     }
 
+    public static void evictPreparedStatement(MD5Digest key, RemovalCause 
cause)

Review Comment:
   :+1:, looking at it again now I just realized this doesn't even need to be 
public, so I'll reduce the visibility to private. 
   
   We ended up only needing this as a separate method to ensure it gets invoked 
in `PstmtPersistenceTest` and apparently byteman is still able to instrument 
around it with private visibility.



-- 
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: pr-unsubscr...@cassandra.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org

Reply via email to