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]