Github user jinmeiliao commented on a diff in the pull request:
https://github.com/apache/incubator-geode/pull/276#discussion_r86836201
--- Diff:
geode-core/src/main/java/org/apache/geode/security/PostProcessor.java ---
@@ -44,6 +47,9 @@ default void init(Properties securityProps) {}
*/
Object processRegionValue(Object principal, String regionName, Object
key, Object value);
+ Collection<Object> processQueryResult(Object principal, Query query,
Collection<String> regions,
--- End diff --
I don't think we want to add this method in the interface. For query,
essentially it's still getting value from region. Customers should not be
burdened with another method to implement if they just want to manipulate the
data. See how query result get post processed in gfsh queries.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
