I don't think we need to add this interface here. Just add it as a method in 
CacheForClientAccess object. The CacheProvider  is an internal interface and is 
only used in the geode-web-api module, we can have it return 
CacheForClientAccess object instead of InternalCache object. This method 
doesn't belong to the regular cache, in my opinion.

Also, the getQueryStore method doesn't need a region path passed to it, right? 
It's always going to be the " "__ParameterizedQueries__" static string. You 
can't pass other string to it to get the query store at all. So we can just do 
this:

  public <K, V> Region<K, V> getQueryStore() {
    Region<K, V> result = delegate.getRegion(PARAMETERIZED_QUERIES_REGION);
    return result;
  }


[ Full content available at: https://github.com/apache/geode/pull/2853 ]
This message was relayed via gitbox.apache.org for 
[email protected]

Reply via email to